Merge pull request #33 from schlessera/as/add-query-args-update-test

Add test for making sure `addQueryArgs` updates existing arguments.
WordPress · Aug 15, 2017 · e2cc1c4 · e2cc1c4
2 parents 0ae3bd3 + ac44be3
commit e2cc1c4
Showing 1 changed file with 7 additions and 0 deletions.
diff --git a/packages/url/src/test/index.test.js b/packages/url/src/test/index.test.js
@@ -17,4 +17,11 @@ describe( 'addQueryArgs', () => {
 
 		expect( addQueryArgs( url, args ) ).toBe( 'https://andalouses.com/beach?night=false&sun=true&sand=false' );
 	} );
+
+	test( 'should update args to an URL with conflicting query string', () => {
+		const url = 'https://andalouses.com/beach?night=false&sun=false&sand=true';
+		const args = { sun: 'true', sand: 'false' };
+
+		expect( addQueryArgs( url, args ) ).toBe( 'https://andalouses.com/beach?night=false&sun=true&sand=false' );
+	} );
 } );