Skip to content

co19/LanguageFeatures/Parts-with-imports/grammar_A04_t01 #2907

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

Closed
scheglov opened this issue Oct 1, 2024 · 5 comments
Closed

co19/LanguageFeatures/Parts-with-imports/grammar_A04_t01 #2907

scheglov opened this issue Oct 1, 2024 · 5 comments
Assignees
Labels
bad-test Report tests in need of updates. When closed, the tests should be considered good

Comments

@scheglov
Copy link

scheglov commented Oct 1, 2024

We don't report an error on the part directive because in principle the target file exists, and even says that it is a part.

FAILED: dart2analyzer-none release_arm64 co19/LanguageFeatures/Parts-with-imports/grammar_A04_t01
Expected: Pass
Actual: MissingCompileTimeError

--- Command "dart2analyzer" (took 33ms):
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=enhanced-parts --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/Parts-with-imports/grammar_A04_t01.dart

static error failures:
- Missing expected unspecified error at tests/co19/src/LanguageFeatures/Parts-with-imports/grammar_A04_t01_part1.dart line 31, column 5, length 4.

--- Re-run this test:
python3 tools/test.py -m release -c dart2analyzer -a arm64 co19/LanguageFeatures/Parts-with-imports/grammar_A04_t01
@scheglov
Copy link
Author

scheglov commented Oct 1, 2024

I'm on the fence about co19/LanguageFeatures/Parts-with-imports/grammar_A03_t01 where we do part of a file that uses part of lib.name. We already report part of lib.name in the part, should we do this also on part? https://dart-review.googlesource.com/c/sdk/+/387820 will start reporting on the part directive.

@sgrekhov sgrekhov self-assigned this Oct 2, 2024
@sgrekhov sgrekhov added the bad-test Report tests in need of updates. When closed, the tests should be considered good label Oct 2, 2024
sgrekhov added a commit to sgrekhov/co19 that referenced this issue Oct 2, 2024
@sgrekhov
Copy link
Contributor

sgrekhov commented Oct 2, 2024

#2912 fixes grammar_A04_t01. For now, with https://dart-review.googlesource.com/c/sdk/+/387820 grammar_A03_t01 passes, so I didn't change it. Please file a separate issue for grammar_A03_t01 if needed.

@eernstg eernstg closed this as completed in d077ffc Oct 2, 2024
@eernstg
Copy link
Member

eernstg commented Oct 2, 2024

I don't know how the error messages about part of lib.name are otherwise positioned, but do note that https://github.com/dart-lang/co19/blob/master/LanguageFeatures/Parts-with-imports/grammar_A04_t01_part1.dart has a syntax error as long as it contains two <partHeader> directives.

@sgrekhov
Copy link
Contributor

sgrekhov commented Oct 2, 2024

Revert. #2913

@scheglov
Copy link
Author

scheglov commented Oct 2, 2024

We (the analyzer team) decided that we don't want to report an error on part, only on part of dotted.name;.

So, it would be good to remove the expectation from co19/LanguageFeatures/Parts-with-imports/grammar_A03_t01.

FAILED: dart2analyzer-none release_arm64 co19/LanguageFeatures/Parts-with-imports/grammar_A03_t01
Expected: Pass
Actual: MissingCompileTimeError

--- Command "dart2analyzer" (took 112ms):
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=enhanced-parts --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/Parts-with-imports/grammar_A03_t01.dart

static error failures:
- Missing expected unspecified error at tests/co19/src/LanguageFeatures/Parts-with-imports/grammar_A03_t01.dart line 31, column 7, length 26.

--- Re-run this test:
python3 tools/test.py -m release -c dart2analyzer -a arm64 co19/LanguageFeatures/Parts-with-imports/grammar_A03_t01

copybara-service bot pushed a commit to dart-lang/sdk that referenced this issue Oct 2, 2024
Maybe? dart-lang/co19#2907

Change-Id: If64ba74bf8d3ad3f5e1b926f19705ae469c76a5b
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/387820
Commit-Queue: Konstantin Shcheglov <scheglov@google.com>
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
sgrekhov added a commit to sgrekhov/co19 that referenced this issue Oct 3, 2024
@eernstg eernstg closed this as completed in a4aeda7 Oct 3, 2024
copybara-service bot pushed a commit to dart-lang/sdk that referenced this issue Oct 4, 2024
2024-10-04 sgrekhov22@gmail.com Fixes dart-lang/co19#2919. Fix new roll failures (dart-lang/co19#2920)
2024-10-03 sgrekhov22@gmail.com dart-lang/co19#2915. Add missing extension in syntax_A03_t01.dart (dart-lang/co19#2918)
2024-10-03 sgrekhov22@gmail.com Fixes dart-lang/co19#2915. Fix errors and typos in some null-aware elements tests (dart-lang/co19#2917)
2024-10-03 sgrekhov22@gmail.com Fixes dart-lang/co19#2906. Add digit separators 'parse' tests (dart-lang/co19#2916)
2024-10-03 sgrekhov22@gmail.com Fixes dart-lang/co19#2907. Update expected errors positions in grammar_A03_t01.dart (dart-lang/co19#2914)
2024-10-02 sgrekhov22@gmail.com dart-lang/co19#2906. Add digit separators tests (dart-lang/co19#2910)
2024-10-02 sgrekhov22@gmail.com dart-lang/co19#2907. Revert wrong change in grammar_A04_t01_part1.dart (dart-lang/co19#2913)
2024-10-02 sgrekhov22@gmail.com Fixes dart-lang/co19#2907. Update expected errors positions in grammar_A04_t01_part1.dart (dart-lang/co19#2912)
2024-10-02 sgrekhov22@gmail.com Fixes dart-lang/co19#2909. Fix extension applicability tests according to the updated spec (dart-lang/co19#2911)
2024-10-01 49699333+dependabot[bot]@users.noreply.github.com Bump the github-actions group with 2 updates (dart-lang/co19#2908)
2024-10-01 sgrekhov22@gmail.com dart-lang/co19#2559. Add tests for adding body to static members (dart-lang/co19#2905)
2024-09-30 sgrekhov22@gmail.com Fixes dart-lang/co19#2902. Add tests for an extension applicability (dart-lang/co19#2903)
2024-09-30 sgrekhov22@gmail.com Fixes dart-lang/co19#2901. Update expected errors for analyzer and CFE (dart-lang/co19#2904)
2024-09-27 sgrekhov22@gmail.com dart-lang/co19#2559. Fix typo in augmenting_enum_values_A05_t05.dart (dart-lang/co19#2900)
2024-09-27 sgrekhov22@gmail.com dart-lang/co19#2559. Add test that `EnumName` extension is not augmentable (dart-lang/co19#2899)
2024-09-27 sgrekhov22@gmail.com dart-lang/co19#2559. Add more tests for augmenting enum members and values. (dart-lang/co19#2897)
2024-09-27 sgrekhov22@gmail.com dart-lang/co19#2825. Add more grammar and terminology tests (dart-lang/co19#2885)
2024-09-27 sgrekhov22@gmail.com dart-lang/co19#2559. Fix accidental syntax errors (dart-lang/co19#2898)

Cq-Include-Trybots: luci.dart.try:analyzer-linux-release-try
Change-Id: Id70baf6e0f6a6ea62eb0b507bffc3c1fb39e71ce
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/388007
Commit-Queue: Erik Ernst <eernst@google.com>
Reviewed-by: Erik Ernst <eernst@google.com>
Reviewed-by: Alexander Thomas <athom@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

3 participants