Add tests for handling mwApiPath and mwWikiPath params in base url di…

…rector

src/MediaWiki.ts

@@ -183,10 +183,6 @@ class MediaWiki {
     this.wikimediaMobileUrlDirector = new WikimediaMobileURLDirector(this.WikimediaMobileApiUrl.href)
   }
 
-  /**
-   * What if wikiPath and apiPath set a bit earlier?
-   * No need to initialize this at the beginning of singleton
-   */
   private initApiURLDirector() {
     this.webUrl = this.baseUrlDirector.buildURL(this.#wikiPath)
     this.apiUrl = this.baseUrlDirector.buildURL(this.#apiPath, this.#wikiPath)

src/util/builders/url/base.director.ts

@@ -13,6 +13,9 @@ export default class BaseURLDirector {
   buildURL(path: string, wikiPath = '') {
     path = this.stripLeadingSlash(path)
     wikiPath = this.stripLeadingSlash(wikiPath)
+    if (wikiPath && !wikiPath.endsWith('/')) {
+      wikiPath += '/'
+    }
     path = `${wikiPath}${path}`
     return urlBuilder.setDomain(this.baseDomain).setPath(path).build(true)
   }

test/unit/builders/url/base.director.test.ts

@@ -9,6 +9,24 @@ describe('BaseURLDirector', () => {
 
       expect(url.href).toBe('https://en.m.wikipedia.com/v1/test/api')
     })
+
+    it('should return URL object with mwApiPath param', () => {
+      const url = baseUrlDirector.buildURL('api.php')
+
+      expect(url.href).toBe('https://en.m.wikipedia.com/api.php')
+    })
+
+    it('should return URL object with mwApiPath ans mwWikiPath params', () => {
+      const url = baseUrlDirector.buildURL('api.php', 'w')
+
+      expect(url.href).toBe('https://en.m.wikipedia.com/w/api.php')
+    })
+
+    it('forward slashes at the beginnig of mwApiPath ans mwWikiPath params should be trimmed', () => {
+      const url = baseUrlDirector.buildURL('/api.php', '/w')
+
+      expect(url.href).toBe('https://en.m.wikipedia.com/w/api.php')
+    })
   })
 
   describe('buildWikimediaApiURL', () => {