Merge pull request #149 from utPLSQL/feature/issue-134-change-pref-defaults

PhilippSalvisberg · web-flow · commit c39ce8006d0c · 2022-02-26T11:24:54.000+01:00
Feature/issue 134 change pref defaults
diff --git a/sqldev/src/main/java/org/utplsql/sqldev/model/preference/PreferenceModel.java b/sqldev/src/main/java/org/utplsql/sqldev/model/preference/PreferenceModel.java
@@ -172,39 +172,39 @@ public void setNumberOfRunsInHistory(final int runs) {
     }
 
     public boolean isShowDisabledCounter() {
-        return getHashStructure().getBoolean(KEY_SHOW_DISABLED_COUNTER, false);
+        return getHashStructure().getBoolean(KEY_SHOW_DISABLED_COUNTER, true);
     }
 
     public void setShowDisabledCounter(final boolean showDisabledCounter) {
         getHashStructure().putBoolean(KEY_SHOW_DISABLED_COUNTER, showDisabledCounter);
     }
 
     public boolean isShowWarningsCounter() {
-        return getHashStructure().getBoolean(KEY_SHOW_WARNINGS_COUNTER, false);
+        return getHashStructure().getBoolean(KEY_SHOW_WARNINGS_COUNTER, true);
     }
 
     public void setShowWarningsCounter(final boolean showWarningCounter) {
         getHashStructure().putBoolean(KEY_SHOW_WARNINGS_COUNTER, showWarningCounter);
     }
 
     public boolean isShowInfoCounter() {
-        return getHashStructure().getBoolean(KEY_SHOW_INFO_COUNTER, false);
+        return getHashStructure().getBoolean(KEY_SHOW_INFO_COUNTER, true);
     }
 
     public void setShowInfoCounter(final boolean showInfoCounter) {
         getHashStructure().putBoolean(KEY_SHOW_INFO_COUNTER, showInfoCounter);
     }
 
     public boolean isShowWarningIndicator() {
-        return getHashStructure().getBoolean(KEY_SHOW_WARNING_INDICATOR, false);
+        return getHashStructure().getBoolean(KEY_SHOW_WARNING_INDICATOR, true);
     }
 
     public void setShowWarningIndicator(final boolean showWarningIndicator) {
         getHashStructure().putBoolean(KEY_SHOW_WARNING_INDICATOR, showWarningIndicator);
     }
 
     public boolean isShowInfoIndicator() {
-        return getHashStructure().getBoolean(KEY_SHOW_INFO_INDICATOR, false);
+        return getHashStructure().getBoolean(KEY_SHOW_INFO_INDICATOR, true);
     }
 
     public void setShowInfoIndicator(final boolean showInfoIndicator) {
@@ -228,7 +228,7 @@ public void setShowDisabledTests(final boolean showDisabledTests) {
     }
 
     public boolean isShowTestDescription() {
-        return getHashStructure().getBoolean(KEY_SHOW_TEST_DESCRIPTION, false);
+        return getHashStructure().getBoolean(KEY_SHOW_TEST_DESCRIPTION, true);
     }
 
     public void setShowTestDescription(final boolean showTestDescription) {
diff --git a/sqldev/src/test/java/org/utplsql/sqldev/test/preference/PreferenceModelTest.java b/sqldev/src/test/java/org/utplsql/sqldev/test/preference/PreferenceModelTest.java
@@ -32,14 +32,14 @@ public void defaultValues() {
         Assert.assertFalse(model.isCheckRunUtplsqlTest());
         Assert.assertFalse(model.isUseSmartTimes());
         Assert.assertEquals(10, model.getNumberOfRunsInHistory());
-        Assert.assertFalse(model.isShowDisabledCounter());
-        Assert.assertFalse(model.isShowWarningsCounter());
-        Assert.assertFalse(model.isShowInfoCounter());
-        Assert.assertFalse(model.isShowWarningIndicator());
-        Assert.assertFalse(model.isShowInfoIndicator());
+        Assert.assertTrue(model.isShowDisabledCounter());
+        Assert.assertTrue(model.isShowWarningsCounter());
+        Assert.assertTrue(model.isShowInfoCounter());
+        Assert.assertTrue(model.isShowWarningIndicator());
+        Assert.assertTrue(model.isShowInfoIndicator());
         Assert.assertTrue(model.isShowSuccessfulTests());
         Assert.assertTrue(model.isShowDisabledTests());
-        Assert.assertFalse(model.isShowTestDescription());
+        Assert.assertTrue(model.isShowTestDescription());
         Assert.assertTrue(model.isSyncDetailTab());
         Assert.assertTrue(model.isShowSuites());
         Assert.assertEquals("test_", model.getTestPackagePrefix());

Original file line number	Diff line number	Diff line change
`@@ -172,39 +172,39 @@ public void setNumberOfRunsInHistory(final int runs) {`
`172`	`172`	`}`
`173`	`173`
`174`	`174`	`public boolean isShowDisabledCounter() {`
`175`		`- return getHashStructure().getBoolean(KEY_SHOW_DISABLED_COUNTER, false);`
	`175`	`+ return getHashStructure().getBoolean(KEY_SHOW_DISABLED_COUNTER, true);`
`176`	`176`	`}`
`177`	`177`
`178`	`178`	`public void setShowDisabledCounter(final boolean showDisabledCounter) {`
`179`	`179`	`getHashStructure().putBoolean(KEY_SHOW_DISABLED_COUNTER, showDisabledCounter);`
`180`	`180`	`}`
`181`	`181`
`182`	`182`	`public boolean isShowWarningsCounter() {`
`183`		`- return getHashStructure().getBoolean(KEY_SHOW_WARNINGS_COUNTER, false);`
	`183`	`+ return getHashStructure().getBoolean(KEY_SHOW_WARNINGS_COUNTER, true);`
`184`	`184`	`}`
`185`	`185`
`186`	`186`	`public void setShowWarningsCounter(final boolean showWarningCounter) {`
`187`	`187`	`getHashStructure().putBoolean(KEY_SHOW_WARNINGS_COUNTER, showWarningCounter);`
`188`	`188`	`}`
`189`	`189`
`190`	`190`	`public boolean isShowInfoCounter() {`
`191`		`- return getHashStructure().getBoolean(KEY_SHOW_INFO_COUNTER, false);`
	`191`	`+ return getHashStructure().getBoolean(KEY_SHOW_INFO_COUNTER, true);`
`192`	`192`	`}`
`193`	`193`
`194`	`194`	`public void setShowInfoCounter(final boolean showInfoCounter) {`
`195`	`195`	`getHashStructure().putBoolean(KEY_SHOW_INFO_COUNTER, showInfoCounter);`
`196`	`196`	`}`
`197`	`197`
`198`	`198`	`public boolean isShowWarningIndicator() {`
`199`		`- return getHashStructure().getBoolean(KEY_SHOW_WARNING_INDICATOR, false);`
	`199`	`+ return getHashStructure().getBoolean(KEY_SHOW_WARNING_INDICATOR, true);`
`200`	`200`	`}`
`201`	`201`
`202`	`202`	`public void setShowWarningIndicator(final boolean showWarningIndicator) {`
`203`	`203`	`getHashStructure().putBoolean(KEY_SHOW_WARNING_INDICATOR, showWarningIndicator);`
`204`	`204`	`}`
`205`	`205`
`206`	`206`	`public boolean isShowInfoIndicator() {`
`207`		`- return getHashStructure().getBoolean(KEY_SHOW_INFO_INDICATOR, false);`
	`207`	`+ return getHashStructure().getBoolean(KEY_SHOW_INFO_INDICATOR, true);`
`208`	`208`	`}`
`209`	`209`
`210`	`210`	`public void setShowInfoIndicator(final boolean showInfoIndicator) {`
`@@ -228,7 +228,7 @@ public void setShowDisabledTests(final boolean showDisabledTests) {`
`228`	`228`	`}`
`229`	`229`
`230`	`230`	`public boolean isShowTestDescription() {`
`231`		`- return getHashStructure().getBoolean(KEY_SHOW_TEST_DESCRIPTION, false);`
	`231`	`+ return getHashStructure().getBoolean(KEY_SHOW_TEST_DESCRIPTION, true);`
`232`	`232`	`}`
`233`	`233`
`234`	`234`	`public void setShowTestDescription(final boolean showTestDescription) {`