Check the browser in a configRoundtrip (#1444)

jenkinsci · Apr 27, 2023 · d7cefd3 · d7cefd3
1 parent 92be490
commit d7cefd3
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/src/test/java/hudson/plugins/git/GitSCMSlowTest.java b/src/test/java/hudson/plugins/git/GitSCMSlowTest.java
@@ -155,10 +155,11 @@ public void testConfigRoundtripExtensionsPreserved() throws Exception {
         assertTrue(scm.getExtensions().toList().contains(localBranchExtension));
 
         /* Save the configuration */
-        rule.configRoundtrip(p);
+        p = rule.configRoundtrip(p);
         List<GitSCMExtension> extensions = scm.getExtensions().toList();
         assertTrue(extensions.contains(localBranchExtension));
         assertEquals("Wrong extension count before reload", 1, extensions.size());
+        rule.assertEqualDataBoundBeans(browser, p.getScm().getBrowser());
 
         /* Reload configuration from disc */
         p.doReload();
@@ -167,6 +168,7 @@ public void testConfigRoundtripExtensionsPreserved() throws Exception {
         assertEquals("Wrong extension count after reload", 1, reloadedExtensions.size());
         LocalBranch reloadedLocalBranch = (LocalBranch) reloadedExtensions.get(0);
         assertEquals(localBranchExtension.getLocalBranch(), reloadedLocalBranch.getLocalBranch());
+        rule.assertEqualDataBoundBeans(browser, reloadedGit.getBrowser());
     }
 
     /*