flutter / flutter-intellij

Flutter Plugin for IntelliJ
https://flutter.dev/using-ide
BSD 3-Clause "New" or "Revised" License
1.98k stars 319 forks source link

Getting "Warning: You should set `SILENT_OBSERVATORY`" on 2020.2 build #4732

Open dpbadiola opened 4 years ago

dpbadiola commented 4 years ago

Hello team, just upgrade my IntelliJ to 2020.2 and am now getting this error on my test packages

Warning: You should set `SILENT_OBSERVATORY` to true when debugging the VM as it will output the observatory URL by default.
This breaks the various reporter contracts.
To set the value define `DART_VM_OPTIONS=-DSILENT_OBSERVATORY=true`.

All test files are working prior to the upgrade. Here are some details am running on

dpbadiola commented 4 years ago

Adding more info after running flutter doctor on a new project

[!] IntelliJ IDEA Ultimate Edition: is partially installed; more components are available. (version 2020.2)

devoncarew commented 4 years ago

Thanks for the report! I assume this is when running a flutter test?

Peng-Qian commented 4 years ago

The same issue, cannot run test after update flutter plugin (IntelliJ - 48.0.4/Android Studio - 48.0.2)

This appears when running any test with the lasted plugin.

Please solve this problem since it heavily strands TDD...

Screen Shot 2020-08-03 at 5 14 01 PM

devoncarew commented 4 years ago

@grouma - we're seeing this when we run flutter test from IDEs. I think we should figure out why we're seeing this and if we should remove the error message from package:test. I can append the full command line that we're starting flutter test up with.

devoncarew commented 4 years ago

It looks like it's just flutter test --machine --start-paused

@Peng-Qian - you might be seeing a separate issue. If so, can you file a separate issue? And, in that issue you might mention what test/target you're running. I see flutter test --machine --start-paused test as your command like - note the trailing test at the end.

mprado-me commented 4 years ago

Hi @devoncarew , thanks for the support

I think that the issue with @Peng-Qian and @dpbadiola are the same as my:

Android Studio 4.0.1 - Build #AI-193.6911.18.40.6626763, built on June 24, 2020 Windows 10 10.0 Flutter Intellij Plugin 48.0.2 Dart Intellij Plugin 193.7361

Basically, after configuring a test run configuration, the tests don't run. It keeps stocked.

Screenshot_82 Screenshot_81

I think that this issue is related with https://github.com/flutter/flutter-intellij/pull/4622

Thanks,

Marco

Peng-Qian commented 4 years ago

My problem is exactly what @marcoprado17 shows... I was trying to upload the same process screenshots...

dpbadiola commented 4 years ago

I can live with the bug by running flutter test on the terminal hope we can get a fixed update soon.

grouma commented 4 years ago

@grouma - we're seeing this when we run flutter test from IDEs. I think we should figure out why we're seeing this and if we should remove the error message from package:test. I can append the full command line that we're starting flutter test up with.

That warning shouldn't be preventing the tests from running. Note the warning was added due to this issue: https://github.com/dart-lang/sdk/issues/31308

I suspect that the issue is related to the start-paused behavior as called out above in https://github.com/flutter/flutter-intellij/issues/4732#issuecomment-668965525

helin24 commented 4 years ago

I've made a temporary fix for directory scope tests hanging; there's some more information in https://github.com/flutter/flutter-intellij/issues/4737.

I am not sure yet what action we want to take on the silent observatory warning.

DanTup commented 4 years ago

I raised an issue in the Flutter repo about this:

https://github.com/flutter/flutter/issues/63075

For VS Code, I haven't seen it preventing the tests from running (we also use --start-paused), but it is confusing. I've been unable to suppress it by following the instructions it prints so I figured it was something that would need handling inside flutter test.

dpbadiola commented 4 years ago

After updating the plugins, everything works now. Thanks team.

Any commit/link that fixed the issue?

campbelg commented 4 years ago

I'm on IntelliJ 2020.2 with Dart Plugin 202.7206, Flutter Plugin 49.0.4, Dart 2.9.2 and Flutter 1.20.3.

I still get the unnecessary 'SILENT_OBSERVATORY' warning when I run a test.