grid - enable smoke and integration tests again

build/tfs/continuous-build.yml

@@ -29,18 +29,18 @@ phases:
   - powershell: |
       .\scripts\test.bat --tfs
     displayName: Run Unit Tests
-  # - powershell: |
-  #     .\scripts\test-integration.bat
-  #   displayName: Run Integration Tests
-  # - powershell: |
-  #     yarn smoketest --screenshots "$(Build.ArtifactStagingDirectory)\artifacts" --log "$(Build.ArtifactStagingDirectory)\artifacts\smoketest.log"
-  #   displayName: Run Smoke Tests
-  # - task: PublishBuildArtifacts@1
-  #   inputs:
-  #     PathtoPublish: '$(Build.ArtifactStagingDirectory)/artifacts'
-  #     ArtifactName: build-artifacts-win32
-  #     publishLocation: Container
-  #   condition: succeededOrFailed()
+  - powershell: |
+      .\scripts\test-integration.bat
+    displayName: Run Integration Tests
+  - powershell: |
+      yarn smoketest --screenshots "$(Build.ArtifactStagingDirectory)\artifacts" --log "$(Build.ArtifactStagingDirectory)\artifacts\smoketest.log"
+    displayName: Run Smoke Tests
+  - task: PublishBuildArtifacts@1
+    inputs:
+      PathtoPublish: '$(Build.ArtifactStagingDirectory)/artifacts'
+      ArtifactName: build-artifacts-win32
+      publishLocation: Container
+    condition: succeededOrFailed()
 
 - phase: Linux
   queue: Hosted Linux Preview
@@ -113,15 +113,15 @@ phases:
   - script: |
       ./scripts/test.sh --tfs
     displayName: Run Unit Tests
-  # - script: |
-  #     ./scripts/test-integration.sh
-  #   displayName: Run Integration Tests
-  # - script: |
-  #     yarn smoketest --screenshots "$(Build.ArtifactStagingDirectory)/artifacts" --log "$(Build.ArtifactStagingDirectory)/artifacts/smoketest.log"
-  #   displayName: Run Smoke Tests
-  # - task: PublishBuildArtifacts@1
-  #   inputs:
-  #     PathtoPublish: '$(Build.ArtifactStagingDirectory)/artifacts'
-  #     ArtifactName: build-artifacts-darwin
-  #     publishLocation: Container
-  #   condition: succeededOrFailed()
\ No newline at end of file
+  - script: |
+      ./scripts/test-integration.sh
+    displayName: Run Integration Tests
+  - script: |
+      yarn smoketest --screenshots "$(Build.ArtifactStagingDirectory)/artifacts" --log "$(Build.ArtifactStagingDirectory)/artifacts/smoketest.log"
+    displayName: Run Smoke Tests
+  - task: PublishBuildArtifacts@1
+    inputs:
+      PathtoPublish: '$(Build.ArtifactStagingDirectory)/artifacts'
+      ArtifactName: build-artifacts-darwin
+      publishLocation: Container
+    condition: succeededOrFailed()
\ No newline at end of file

src/vs/workbench/api/electron-browser/mainThreadEditors.ts

@@ -54,6 +54,7 @@ export class MainThreadTextEditors implements MainThreadTextEditorsShape {
 		this._toDispose.push(documentsAndEditors.onTextEditorRemove(editors => editors.forEach(this._onTextEditorRemove, this)));
 
 		this._toDispose.push(this._editorService.onDidVisibleEditorsChange(() => this._updateActiveAndVisibleTextEditors()));
+		this._toDispose.push(this._editorGroupService.onDidRemoveGroup(() => this._updateActiveAndVisibleTextEditors()));
 
 		this._registeredDecorationTypes = Object.create(null);
 	}