Skip to content

Commit 3215b17

Browse files
committed
test: restore preference conflict status indicator coverage
Check a visible warning or repair indicator for conflicting preference files and a valid indicator for a clean JSON file.
1 parent c4d4bdb commit 3215b17

1 file changed

Lines changed: 13 additions & 2 deletions

File tree

‎test/spec/PreferencesManager-integ-test.js‎

Lines changed: 13 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -156,8 +156,12 @@ define(function (require, exports, module) {
156156

157157
// there will be an error in problems panel if both present
158158
await awaitsForDone(SpecRunnerUtils.openProjectFiles(".phcode.json"));
159-
// Check the diagnostic itself: another provider can offer a fix, which changes
160-
// the status icon to inspection-repair while the preference conflict still exists.
159+
// Keep the visual assertion: another provider can offer a fix, which changes
160+
// the warning icon to a repair icon while the preference conflict still exists.
161+
await awaitsFor(()=>{
162+
return testWindow.$("#status-inspection").is(":visible") &&
163+
testWindow.$("#status-inspection").is(".inspection-errors, .inspection-repair");
164+
}, "visible problem indicator on .phcode.json");
161165
if (!testWindow.$("#problems-panel").is(":visible")) {
162166
CommandManager.execute(Commands.VIEW_TOGGLE_PROBLEMS);
163167
}
@@ -169,11 +173,18 @@ define(function (require, exports, module) {
169173
await awaitsFor(()=>{
170174
return !testWindow.$("#problems-panel").text().includes(Strings.ERROR_PREFS_PROJECT_LINT_MESSAGE);
171175
}, "no preference conflict for normal test.json file");
176+
await awaitsFor(()=>{
177+
return testWindow.$("#status-inspection").is(".inspection-valid:visible");
178+
}, "valid inspection indicator for normal test.json file");
172179

173180
await awaitsForDone(SpecRunnerUtils.openProjectFiles(".brackets.json"));
174181
await awaitsFor(()=>{
175182
return testWindow.$("#problems-panel").text().includes(Strings.ERROR_PREFS_PROJECT_LINT_MESSAGE);
176183
}, "problem panel on .brackets.json");
184+
await awaitsFor(()=>{
185+
return testWindow.$("#status-inspection").is(":visible") &&
186+
testWindow.$("#status-inspection").is(".inspection-errors, .inspection-repair");
187+
}, "visible problem indicator on .brackets.json");
177188
});
178189

179190
it("should open .brackets.json file if it has json errors", async function () {

0 commit comments

Comments
 (0)