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

co19/LanguageFeatures/Augmentation-libraries/augmented_expression_A01_t05 #2660

Closed
scheglov opened this issue May 13, 2024 · 2 comments
Closed
Assignees
Labels
bad-test Report tests in need of updates. When closed, the tests should be considered good

Comments

@scheglov
Copy link

Looks like Expect is not imported.
And topLevelVariable is not declared.

FAILED: dart2analyzer-none release_arm64 co19/LanguageFeatures/Augmentation-libraries/augmented_expression_A01_t05
Expected: Pass
Actual: CompileTimeError

--- Command "dart2analyzer" (took 08.000250s):
DART_CONFIGURATION=ReleaseARM64 sdk/bin/dart --packages=.dart_tool/package_config.json pkg/analyzer_cli/bin/analyzer.dart --use-analysis-driver-memory-byte-store --dart-sdk=/Users/scheglov/Source/Dart/sdk.git/sdk/sdk -Dtest_runner.configuration=custom-configuration-1 --enable-experiment=macros --ignore-unrecognized-flags --packages=/Users/scheglov/Source/Dart/sdk.git/sdk/.dart_tool/package_config.json --format=json /Users/scheglov/Source/Dart/sdk.git/sdk/tests/co19/src/LanguageFeatures/Augmentation-libraries/augmented_expression_A01_t05.dart

unexpected analysis errors in augmented_expression_A01_t05.dart:
- Line 55, column 3: COMPILE_TIME_ERROR.UNDEFINED_IDENTIFIER
  Undefined name 'Expect'.

- Line 56, column 3: COMPILE_TIME_ERROR.UNDEFINED_IDENTIFIER
  Undefined name 'Expect'.

- Line 57, column 3: COMPILE_TIME_ERROR.UNDEFINED_IDENTIFIER
  Undefined name 'Expect'.

- Line 58, column 3: COMPILE_TIME_ERROR.UNDEFINED_IDENTIFIER
  Undefined name 'Expect'.

- Line 59, column 3: COMPILE_TIME_ERROR.UNDEFINED_IDENTIFIER
  Undefined name 'Expect'.

- Line 60, column 3: COMPILE_TIME_ERROR.UNDEFINED_IDENTIFIER
  Undefined name 'Expect'.

- Line 61, column 3: COMPILE_TIME_ERROR.UNDEFINED_IDENTIFIER
  Undefined name 'Expect'.

- Line 62, column 3: COMPILE_TIME_ERROR.UNDEFINED_IDENTIFIER
  Undefined name 'Expect'.

- Line 63, column 3: COMPILE_TIME_ERROR.UNDEFINED_IDENTIFIER
  Undefined name 'Expect'.

unexpected analysis errors in augmented_expression_A01_t05_lib.dart:
- Line 20, column 1: COMPILE_TIME_ERROR.AUGMENTATION_WITHOUT_DECLARATION
  The declaration being augmented doesn't exist.

- Line 21, column 3: COMPILE_TIME_ERROR.UNDEFINED_IDENTIFIER
  Undefined name 'Expect'.

- Line 25, column 3: COMPILE_TIME_ERROR.UNDEFINED_IDENTIFIER
  Undefined name 'Expect'.

- Line 31, column 5: COMPILE_TIME_ERROR.UNDEFINED_IDENTIFIER
  Undefined name 'Expect'.

- Line 35, column 5: COMPILE_TIME_ERROR.UNDEFINED_IDENTIFIER
  Undefined name 'Expect'.

- Line 39, column 5: COMPILE_TIME_ERROR.UNDEFINED_IDENTIFIER
  Undefined name 'Expect'.

- Line 43, column 5: COMPILE_TIME_ERROR.UNDEFINED_IDENTIFIER
  Undefined name 'Expect'.

- Line 50, column 5: COMPILE_TIME_ERROR.UNDEFINED_IDENTIFIER
  Undefined name 'Expect'.

- Line 54, column 5: COMPILE_TIME_ERROR.UNDEFINED_IDENTIFIER
  Undefined name 'Expect'.

- Line 58, column 5: COMPILE_TIME_ERROR.UNDEFINED_IDENTIFIER
  Undefined name 'Expect'.

- Line 62, column 5: COMPILE_TIME_ERROR.UNDEFINED_IDENTIFIER
  Undefined name 'Expect'.

- Line 71, column 5: COMPILE_TIME_ERROR.UNDEFINED_IDENTIFIER
  Undefined name 'Expect'.

- Line 75, column 5: COMPILE_TIME_ERROR.UNDEFINED_IDENTIFIER
  Undefined name 'Expect'.

- Line 79, column 5: COMPILE_TIME_ERROR.UNDEFINED_IDENTIFIER
  Undefined name 'Expect'.

- Line 83, column 5: COMPILE_TIME_ERROR.UNDEFINED_IDENTIFIER
  Undefined name 'Expect'.

- Line 90, column 5: COMPILE_TIME_ERROR.UNDEFINED_IDENTIFIER
  Undefined name 'Expect'.

- Line 94, column 5: COMPILE_TIME_ERROR.UNDEFINED_IDENTIFIER
  Undefined name 'Expect'.

- Line 98, column 5: COMPILE_TIME_ERROR.UNDEFINED_IDENTIFIER
  Undefined name 'Expect'.

- Line 102, column 5: COMPILE_TIME_ERROR.UNDEFINED_IDENTIFIER
  Undefined name 'Expect'.

--- Re-run this test:
python3 tools/test.py -m release -c dart2analyzer -a arm64 co19/LanguageFeatures/Augmentation-libraries/augmented_expression_A01_t05
@sgrekhov sgrekhov self-assigned this May 14, 2024
sgrekhov added a commit to sgrekhov/co19 that referenced this issue May 14, 2024
@sgrekhov sgrekhov added the bad-test Report tests in need of updates. When closed, the tests should be considered good label May 14, 2024
copybara-service bot pushed a commit to dart-lang/sdk that referenced this issue May 17, 2024
2024-05-16 sgrekhov22@gmail.com dart-lang/co19#2641. Rename `unchanged` -> `other_declarations` (dart-lang/co19#2670)
2024-05-16 sgrekhov22@gmail.com dart-lang/co19#2641. Check that it is still an error to declare more than one class member named `_` (dart-lang/co19#2669)
2024-05-15 sgrekhov22@gmail.com dart-lang/co19#2641. Add tests for multiple top-level declarations named `_` (dart-lang/co19#2667)
2024-05-15 sgrekhov22@gmail.com dart-lang/co19#2641. Add tests for unchanged type declarations named `_` (dart-lang/co19#2666)
2024-05-14 sgrekhov22@gmail.com dart-lang/co19#2641. Add tests for unchanged declarations named `_` (dart-lang/co19#2665)
2024-05-14 sgrekhov22@gmail.com dart-lang/co19#2641. Add tests for multiple declarations named `_` (dart-lang/co19#2656)
2024-05-14 sgrekhov22@gmail.com Fixes dart-lang/co19#2661. Fix numerous typos in augmented_expression_A01_t09 (dart-lang/co19#2664)
2024-05-14 sgrekhov22@gmail.com Fixes dart-lang/co19#2659. Make unique extension members names in augmenting_types_A05_t04 (dart-lang/co19#2662)
2024-05-14 sgrekhov22@gmail.com Fixes dart-lang/co19#2660. Fix typos in augmented_expression_A01_t05 (dart-lang/co19#2663)
2024-05-14 sgrekhov22@gmail.com dart-lang/co19#2641. Add wildcard initializer tests (dart-lang/co19#2658)
2024-05-13 sgrekhov22@gmail.com dart-lang/co19#2119. Remove some accidental comments (dart-lang/co19#2657)

Change-Id: I665f56c1039b14354019a32b94183f6d9c9bc5c7
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/367020
Reviewed-by: Erik Ernst <eernst@google.com>
Commit-Queue: Alexander Thomas <athom@google.com>
Reviewed-by: Alexander Thomas <athom@google.com>
@scheglov
Copy link
Author

I still see errors because Expect is not imported into augmented_expression_A01_t05_lib.dart

@scheglov scheglov reopened this May 17, 2024
@sgrekhov
Copy link
Contributor

After rereading of the spec

Likewise, libraries imported by the augmented library are not implicitly imported by the library augmentation.

The test must be really fixed, missed this, sorry!

sgrekhov added a commit to sgrekhov/co19 that referenced this issue May 17, 2024
sgrekhov added a commit to sgrekhov/co19 that referenced this issue May 17, 2024
copybara-service bot pushed a commit to dart-lang/sdk that referenced this issue May 24, 2024
2024-05-24 sgrekhov22@gmail.com Fixes dart-lang/co19#2684. Update expected errors positions for CFE (dart-lang/co19#2685)
2024-05-24 sgrekhov22@gmail.com dart-lang/co19#2641. Add wildcards do not shadow tests (dart-lang/co19#2683)
2024-05-23 sgrekhov22@gmail.com dart-lang/co19#2641. Add patterns tests (dart-lang/co19#2682)
2024-05-22 sgrekhov22@gmail.com dart-lang/co19#2641. Check that it is still an error to declare more than one enum member named `_` (dart-lang/co19#2674)
2024-05-22 sgrekhov22@gmail.com dart-lang/co19#2641. Add import prefix tests (dart-lang/co19#2681)
2024-05-22 sgrekhov22@gmail.com dart-lang/co19#2641. Add record types tests (dart-lang/co19#2679)
2024-05-22 sgrekhov22@gmail.com dart-lang/co19#2641. Add local functions test (dart-lang/co19#2680)
2024-05-22 sgrekhov22@gmail.com dart-lang/co19#2641. Check that it is still an error to declare more than one extension type member named `_` (dart-lang/co19#2675)
2024-05-21 sgrekhov22@gmail.com dart-lang/co19#2641. Add late variables named `_` tests (dart-lang/co19#2677)
2024-05-21 sgrekhov22@gmail.com dart-lang/co19#2641. Check that it is still an error to declare more than one extension member named _ (dart-lang/co19#2676)
2024-05-19 sgrekhov22@gmail.com Fixes dart-lang/co19#2660. Add missing import to augmented_expression_A01_t05_lib.dart (dart-lang/co19#2673)
2024-05-17 sgrekhov22@gmail.com dart-lang/co19#2641. Check that it is still an error to declare more than one mixin member named `_` (dart-lang/co19#2672)
2024-05-17 sgrekhov22@gmail.com dart-lang/co19#2641. Wildcards. Update expected errors positions for CFE (dart-lang/co19#2671)

Change-Id: Ia94383cb2dc2a5912950ee3e13ee93a9576f3500
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/367963
Reviewed-by: Alexander Thomas <athom@google.com>
Reviewed-by: Erik Ernst <eernst@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bad-test Report tests in need of updates. When closed, the tests should be considered good
Projects
None yet
Development

No branches or pull requests

2 participants