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

Bump analyzer to 2.4.0 #223

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

GZGavinZhao
Copy link

This makes pageloader compatible with AngularDart 7+.

However, currently this adds two more errors to the test:

Before:

01:05 +376 -1: test/webdriver_typing_test.dart: clear with focus and blur with empty text without focusBefore and blurAfter [E]                                                                                                              
  Expected: <1>
    Actual: <0>
  
  package:test_api            expect
  test/src/typing.dart 147:9  runTests.<fn>.<fn>.<fn>

Now:

01:09 +396 -1: test/webdriver_typing_test.dart: clear with focus and blur with empty text without focusBefore and blurAfter [E]                                                                                                              
  Expected: <0>
    Actual: <1>
  
  package:test_api            expect
  test/src/typing.dart 153:9  runTests.<fn>.<fn>.<fn>
  
01:12 +404 -2: test/webdriver_typing_test.dart: clear with focus and blur with non-empty text without focusBefore and blurAfter [E]                                                                                                          
  Expected: <2>
    Actual: <1>
  
  package:test_api            expect
  test/src/typing.dart 177:9  runTests.<fn>.<fn>.<fn>
  
01:22 +446 -3: test/webdriver_typing_null_safety_test.dart: typing with focus and blur without focusBefore and blurAfter [E]                                                                                                                 
  Expected: <1>
    Actual: <0>
  
  package:test_api                       expect
  test/src_null_safety/typing.dart 98:7  runTests.<fn>.<fn>
  
01:27 +459 -3: Some tests failed. 

Signed-off-by: GZGavinZhao <[email protected]>
@GZGavinZhao
Copy link
Author

Oh that was because I ran test directly with dart test.

If I use ./tool/travis.sh, the test suite passes with no errors.

@GZGavinZhao GZGavinZhao marked this pull request as ready for review January 10, 2022 04:21
Use the `@Tags` to differentiate between core, creator, webdriver, and
browser tests. In this way, there's no need to invoke `tool/travis.sh`
and developers can easily choose which tests they want to run (e.g.
`dart test -x html`, `dart test -t webdriver`, etc.).

Signed-off-by: Gavin Zhao <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant