Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Generic Test references VM specific hidden fields #1959

Closed
DartBot opened this issue Mar 3, 2012 · 2 comments
Closed

Generic Test references VM specific hidden fields #1959

DartBot opened this issue Mar 3, 2012 · 2 comments
Assignees
Labels
area-vm Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends.

Comments

@DartBot
Copy link

DartBot commented Mar 3, 2012

This issue was originally filed by zundel@google.com


tests/language/src/GenericTest.dart

Its trying to determine the line number of the place where the error
was thrown, but the field being used is VM specific.

file:/home/zundel/dart/tests/language/src/GenericTest.dart:30: C.T is not assignable to A
    29: class C<T> {
    30: B<T> b_;
file:/home/zundel/dart/tests/language/src/GenericTest.dart:31: C.T is not assignable to A
    30: B<T> b_;
    31: C(T t) : b_ = new B<T>(t) { }
file:/home/zundel/dart/tests/language/src/GenericTest.dart:64: "dstName" is not a member of TypeError
    63: Expect.equals("AX", error.srcType);
    64: Expect.equals("a_", error.dstName);
file:/home/zundel/dart/tests/language/src/GenericTest.dart:65: "url" is not a member of TypeError
    64: Expect.equals("a_", error.dstName);
    65: int pos = error.url.lastIndexOf("/", error.url.length);
file:/home/zundel/dart/tests/language/src/GenericTest.dart:65: "url" is not a member of TypeError
    64: Expect.equals("a_", error.dstName);
    65: int pos = error.url.lastIndexOf("/", error.url.length);
file:/home/zundel/dart/tests/language/src/GenericTest.dart:67: "url" is not a member of TypeError
    66: if (pos == -1) {
    67: pos = error.url.lastIndexOf("&quot;, error.url.length);
file:/home/zundel/dart/tests/language/src/GenericTest.dart:67: "url" is not a member of TypeError
    66: if (pos == -1) {
    67: pos = error.url.lastIndexOf("&quot;, error.url.length);
file:/home/zundel/dart/tests/language/src/GenericTest.dart:69: "url" is not a member of TypeError
    68: }
    69: String subs = error.url.substring(pos + 1, error.url.length);
file:/home/zundel/dart/tests/language/src/GenericTest.dart:69: "url" is not a member of TypeError
    68: }
    69: String subs = error.url.substring(pos + 1, error.url.length);
file:/home/zundel/dart/tests/language/src/GenericTest.dart:71: "line" is not a member of TypeError
    70: Expect.equals("GenericTest.dart", subs);
    71: Expect.equals(23, error.line);
file:/home/zundel/dart/tests/language/src/GenericTest.dart:72: "column" is not a member of TypeError
    71: Expect.equals(23, error.line);
    72: Expect.equals(23, error.column);
zundel@santafe(analysis-batch-3):/dart$ ./tools/test.py --component dartc tests/language/src/GenericTest.dart
zundel@santafe(analysis-batch-3):
/dart$ ./tools/test.py --component dartc language/GenericTest.dart
[00:01 | 100% | + 1 | - 0]

zundel@santafe(analysis-batch-3):~/dart$ frog tests/language/src/GenericTest.dart
tests/language/src/GenericTest.dart:64:27: warning: cannot find "dstName" on "TypeError"
      Expect.equals("a_", error.dstName);
                          ^^^^^^^^^^^^^
tests/language/src/GenericTest.dart:20:9: warning: type AX is not a subtype of A
class B<T extends A> {
        ^^^^^^^^^^^
/tmp/tmpA40rVy/out.js:14: Expect.equals(expected: <1>, actual: <0>) fails.
  throw e;

file:/home/zundel/dart/tests/language/src/GenericTest.dart:64: "dstName" is not a member of TypeError
    63: Expect.equals("AX", error.srcType);
    64: Expect.equals("a_", error.dstName);
file:/home/zundel/dart/tests/language/src/GenericTest.dart:65: "url" is not a member of TypeError
    64: Expect.equals("a_", error.dstName);
    65: int pos = error.url.lastIndexOf("/", error.url.length);
file:/home/zundel/dart/tests/language/src/GenericTest.dart:65: "url" is not a member of TypeError
    64: Expect.equals("a_", error.dstName);
    65: int pos = error.url.lastIndexOf("/", error.url.length);
file:/home/zundel/dart/tests/language/src/GenericTest.dart:67: "url" is not a member of TypeError
    66: if (pos == -1) {
    67: pos = error.url.lastIndexOf("&quot;, error.url.length);
file:/home/zundel/dart/tests/language/src/GenericTest.dart:67: "url" is not a member of TypeError
    66: if (pos == -1) {
    67: pos = error.url.lastIndexOf("&quot;, error.url.length);
file:/home/zundel/dart/tests/language/src/GenericTest.dart:69: "url" is not a member of TypeError
    68: }
    69: String subs = error.url.substring(pos + 1, error.url.length);
file:/home/zundel/dart/tests/language/src/GenericTest.dart:69: "url" is not a member of TypeError
    68: }
    69: String subs = error.url.substring(pos + 1, error.url.length);
file:/home/zundel/dart/tests/language/src/GenericTest.dart:71: "line" is not a member of TypeError
    70: Expect.equals("GenericTest.dart", subs);
    71: Expect.equals(23, error.line);
file:/home/zundel/dart/tests/language/src/GenericTest.dart:72: "column" is not a member of TypeError
    71: Expect.equals(23, error.line);
    72: Expect.equals(23, error.column);

I don't think Frog has a chance of passing this test. Can it be moved to a Vm specific dir?

zundel@santafe(analysis-batch-3):~/dart$ frog tests/language/src/GenericTest.dart
tests/language/src/GenericTest.dart:64:27: warning: cannot find "dstName" on "TypeError"
      Expect.equals("a_", error.dstName);
                          ^^^^^^^^^^^^^
tests/language/src/GenericTest.dart:20:9: warning: type AX is not a subtype of A
class B<T extends A> {
        ^^^^^^^^^^^
/tmp/tmpA40rVy/out.js:14: Expect.equals(expected: <1>, actual: <0>) fails.
  throw e;
        ^
Expect.equals(expected: <1>, actual: <0>) fails.
    at Function._fail (/tmp/tmpA40rVy/out.js:178:3)
    at Function.equals (/tmp/tmpA40rVy/out.js:172:10)
    at Function.testMain (/tmp/tmpA40rVy/out.js:642:10)
    at main (/tmp/tmpA40rVy/out.js:646:15)
    at /tmp/tmpA40rVy/out.js:656:1

(If we fix the test and put @­static-clean on this test, we should get a failure from the analyzer if this were to happen again)

@iposva-google
Copy link
Contributor

Set owner to @crelier.
Added Accepted label.

@crelier
Copy link
Contributor

crelier commented Apr 3, 2012

Fixed in r6149.


Added Fixed label.

@DartBot DartBot added Type-Defect area-vm Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends. labels Apr 3, 2012
copybara-service bot pushed a commit that referenced this issue Feb 20, 2023
…ath, pool, source_maps, test, tools, webdev

Revisions updated by `dart tools/rev_sdk_deps.dart`.

args (https://github.com/dart-lang/args/compare/b08471e..9305d5a):
  9305d5a  2023-02-15  Naoto Kaneko  Update README.md (#236)

characters (https://github.com/dart-lang/characters/compare/29f3a3e..3281cc7):
  3281cc7  2023-02-16  Devon Carew  blast_repo fixes (#75)

dartdoc (https://github.com/dart-lang/dartdoc/compare/3931595..dba6f94):
  dba6f946  2023-02-16  Janice Collins  Preserve type aliases for records in output (#3340)
  96e61903  2023-02-16  Sam Rawlins  Remove unused additionalOptions parameter, other little refactorings (#3339)
  f64290e1  2023-02-13  Devon Carew  remove an unused css attribute (#3335)
  c055526a  2023-02-13  Devon Carew  fix an issue where we created a temporary directory when performing a dry run (#3334)
  1f6c8db5  2023-02-13  dependabot[bot]  Bump github/codeql-action from 2.2.1 to 2.2.4 (#3337)
  f1fa0efc  2023-02-13  dependabot[bot]  Bump actions/cache from 3.2.4 to 3.2.5 (#3338)

ffi (https://github.com/dart-lang/ffi/compare/69d7596..32f5eef):
  32f5eef  2023-02-20  Daco Harkes  Bump SDK constraint to 4.0.0 (#185)

http (https://github.com/dart-lang/http/compare/f4b365e..c13a3f8):
  c13a3f8  2023-02-17  Alex Li  ✨ Add Cronet embedded tool (#853)

markdown (https://github.com/dart-lang/markdown/compare/f51c24c..4befe66):
  4befe66  2023-02-16  Zhiguang Chen  Do not generate heading IDs for headings with no content in `HeaderWithIdSyntax` (#522)
  cae08af  2023-02-15  Kevin Moore  Prepare to release v7.0.1 (#520)
  bdbaf76  2023-02-15  Zhiguang Chen  Remove lookarounds from autolink extension patterns (#519)
  eb09fac  2023-02-14  Kevin Moore  Move to pkg:dart_flutter_team_lints (#518)
  51a6389  2023-02-15  Zhiguang Chen  Add line endings to HTML blocks (#512)
  a4cd7a0  2023-02-14  Kevin Moore  Label tests that only run on the VM: allows running tests with browser (#516)
  6788042  2023-02-14  Kevin Moore  CI: add publish and response workflows (#517)

mockito (https://github.com/dart-lang/mockito/compare/d2a8df1..ed5bd84):
  ed5bd84  2023-01-09  Ross Wang  Relax mixin criteria
  781752c  2023-02-13  Sam Rawlins  Change `void` to `dynamic` when overriding method arguments
  71e41b9  2023-02-10  Sam Rawlins  Add `ignore_for_file: use_of_void_result`
  d5a25f8  2023-02-08  Sam Rawlins  Fix violations of `unnecessary_parenthesis` lint

path (https://github.com/dart-lang/path/compare/a95f1e9..24b58a2):
  24b58a2  2023-02-20  Kevin Moore  Move to team lints, require Dart 2.19 (#138)
  8ec8ca0  2023-02-18  Kevin Moore  blast_repo fixes (#137)

pool (https://github.com/dart-lang/pool/compare/51f1131..694cfd8):
  694cfd8  2023-02-20  Kevin Moore  move to package:dart_flutter_team_lints, require Dart 2.19 (#65)
  40bf2af  2023-02-20  Kevin Moore  blast_repo fixes (#64)

source_maps (https://github.com/dart-lang/source_maps/compare/cf44db3..a112e98):
  a112e98  2023-02-16  Devon Carew  configure publishing automation (#74)

test (https://github.com/dart-lang/test/compare/b5e70db..e56c643):
  e56c6439  2023-02-17  Jacob MacDonald  Add exe compiler, supports running tests compiled to native executables (#1941)
  f80dfa00  2023-02-16  Nate Bosch  Add note about why we are replacing matcher (#1940)
  732ae1de  2023-02-15  Nate Bosch  Add `because` usage in examples (#1939)
  f6df9756  2023-02-15  Jacob MacDonald  Fix typo in readme (#1937)
  aca53419  2023-02-14  Nate Bosch  Add a caret constraint for matcher (#1927)
  f7f3a019  2023-02-14  Jacob MacDonald  use test_api/backend.dart instead of test_core/backend.dart (#1936)
  de40c1c0  2023-02-14  Jacob MacDonald  Add support for `--compiler` flag (#1903)
  1f42db8c  2023-02-13  Devon Carew  updates to package:checks docs (#1933)

tools (https://github.com/dart-lang/tools/compare/48a544b..a53933c):
  a53933c  2023-02-17  Elias Yishak  [package:dash_analytics] Tests for conforming to GA4 Measurement Protocol limitations (#9)
  c54430b  2023-02-17  Devon Carew  add some clarification to the top-level readme (#8)

webdev (https://github.com/dart-lang/webdev/compare/0bae2be..ae7eb80):
  ae7eb80  2023-02-17  Anna Gringauze  Add tests for object inspection (#1973)
  7d80a2c  2023-02-17  Elliott Brooks (she/her)  Ignore offset / count if an instance has no `length`  (#1972)
  cd66172  2023-02-17  Elliott Brooks (she/her)  [MV3 Debug Extension] User can reload app and continue to debug (#1968)
  8b7f9d3  2023-02-16  Anna Gringauze  Re-enable weak webdev tests (#1960)
  83d8e47  2023-02-16  Elliott Brooks (she/her)  [MV3 Debug Extension] Variables in panel.dart should be private (#1969)
  4e85e74  2023-02-16  Elliott Brooks (she/her)  [MV3 Debug Extension] The new debug extension can be run on Manifest V3 or Manifest V2 (#1966)
  3982f5f  2023-02-15  Elliott Brooks (she/her)  [MV3 Debug Extension] Clean up tests in preparation for supporting compiling to MV2  (#1964)
  3ad544e  2023-02-15  Elliott Brooks (she/her)  [MV3 Debug Extension] Remove isMV3Extension field from DevtoolsRequest (#1963)
  c77043e  2023-02-14  Elliott Brooks (she/her)  [MV3 Debug Extension] Fix isDevMode getter (#1962)
  a761125  2023-02-14  Elliott Brooks (she/her)  [MV3 Debug Extension] Print `console` messages on test failure (#1961)
  41e92be  2023-02-14  Anna Gringauze  Create test_common package (#1945)
  443f820  2023-02-14  Anna Gringauze  Re-enable skipped reload test (#1958)
  a7bc3fc  2023-02-14  Anna Gringauze  Make dart-uri-file-uri test use sound null safety (#1959)
  464a8eb  2023-02-14  Elliott Brooks (she/her)  [MV3 Debug Extension] Compile extension with Dart instead of shell script (#1954)
  7cf8fe2  2023-02-13  Anna Gringauze  Fix asset handler tests (#1956)
  c371ad7  2023-02-13  Anna Gringauze  Fix chrome_proxy_service tests broken after switch to null safety by default (#1957)
  d6ec127  2023-02-13  Anna Gringauze  Move test-only code from the SdkLayout (#1955)
  35fa34b  2023-02-13  Elliott Brooks (she/her)  [MV3 Debug Extension] Update Chrome APIs to be backwards compatible with MV2 (#1951)

Change-Id: I4250594d0fa775d07cb3a0f8594e634552de452b
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/284240
Reviewed-by: Konstantin Shcheglov <scheglov@google.com>
Auto-Submit: Devon Carew <devoncarew@google.com>
Commit-Queue: Konstantin Shcheglov <scheglov@google.com>
copybara-service bot pushed a commit that referenced this issue Feb 27, 2023
Revisions updated by `dart tools/rev_sdk_deps.dart`.

cli_util (https://github.com/dart-lang/cli_util/compare/fd38b5f..3851652):
  3851652  2023-02-24  Kevin Moore  Move to dart_flutter_team_lints, require Dart 2.19 (#77)
  705bb29  2023-02-24  Kevin Moore  blast_repo fixes (#76)

crypto (https://github.com/dart-lang/crypto/compare/03eb2c9..9efb888):
  9efb888  2023-02-23  Kevin Moore  move to pkg:dart_flutter_team_lints, require Dart 2.19 (#141)

test (https://github.com/dart-lang/test/compare/2cc4144..1307cc5):
  1307cc59  2023-02-23  Nate Bosch  Add ComparableChecks extension (#1960)
  66f2985c  2023-02-24  Pascal Welsch  Fix isNotEmpty rejection (#1959)
  cf10dc5b  2023-02-23  Nate Bosch  Condition must have dynamic generic in deepEquals (#1958)
  6fc48d03  2023-02-23  Nate Bosch  Fix timeouts when a predicate throws instead (#1957)

vector_math (https://github.com/google/vector_math.dart/compare/920e9b6..1e4d000):
  1e4d000  2023-02-23  Kevin Moore  Move to latest team lints, require Dart 2.19 (#281)

webdev (https://github.com/dart-lang/webdev/compare/1e7f9b7..deb801b):
  deb801b  2023-02-24  Devon Carew  contribute issue templates and pull request labeling (#1979)
  74a6520  2023-02-24  Elliott Brooks (she/her)  [MV3 Debug Extension] Use new Dart Debug Extension bug template (#1992)
  c1f325c  2023-02-24  Elliott Brooks (she/her)  [MV3 Debug Extension] Bolt debugging: can reload the page and continue to debug (#1991)
  057f598  2023-02-23  Elliott Brooks (she/her)  [MV3 Debug Extension] Small fixes for building the extension (#1990)
  28a090e  2023-02-23  Anna Gringauze  Run dwds tests in a copy of the sdk directory (#1989)
  b58d3eb  2023-02-23  Anna Gringauze  Run webdev tests in a copy of the sdk directory (#1987)

Change-Id: Iaad67eec68cf3cfeba832abbeb42728043e15b22
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/285760
Auto-Submit: Devon Carew <devoncarew@google.com>
Commit-Queue: Konstantin Shcheglov <scheglov@google.com>
Reviewed-by: Konstantin Shcheglov <scheglov@google.com>
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-vm Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends.
Projects
None yet
Development

No branches or pull requests

3 participants