TestContainersUpdated events are not sent on project open/close
@armanio123 đang làm issue này rồi.
Từ ngày 27/6/2019.
Đánh giá
Issue này chưa được đánh giá.
Mô tả
Expected Behavior
The TestContainerDiscoverer class is responsible for notifying the TestWindow of containers where tests should be discovered. It should call the TestContainersUpdated event whenever tests could potentially have changed.
My expectation is that OnProjectLoaded and OnProjectUnloaded should send the TestContainersUpdated event.
Actual Behavior
If you would put a breakpoint you can see that the OnTestContainersChanged is called, but the if condition evaluates to false and the event isn't actually sent, meaning that tests don't get removed on project close and don't get discovered on project open.
- NTVS Version: 1.5.10610.1 Commit Hash:529a87de2769e143655e31ea81795c42e5775ef8
- Visual Studio Version: 16.2 preview 4
- Node.js Version: 8.1.4
Steps to Reproduce
- Open a nodejs project
- Run build to discover tests
- Unload the project
- If the event would have been sent successfully the tests for the unloaded project should disappear
In the OnProjectUnloaded method the project is removed from knownProjects and this makes the if condition always fail, whereas in the OnProjectLoaded method the newly created project has HasRequestedContainers set to false making the if condition always fail.
- Ngôn ngữ chính
- C#
- Star
- 1.8k
- Fork
- 355
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của microsoft/nodejstools
-
microsoft/nodejstools#2523 · 1 reaction · 1 người được giao ·
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 25/100
microsoft/nodejstools#2519 · 1 bình luận ·
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 15/100
microsoft/nodejstools#2517 ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 25/100
microsoft/nodejstools#2514 ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 35/100
microsoft/nodejstools#2513 · 5 bình luận ·
Tất cả issue của microsoft/nodejstools
Issue tương tự
-
type/automation type/tech-debt
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
-
t/bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
-
ci-failure-cause test-failure
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
-
area:auth FE mvp P3
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
klasolsson81/jobbliggaren#1788 ·