detail page: add country to postcode -search by name- link

[nominatim-ui.git] / test / details.js

diff --git a/test/details.js b/test/details.js
index fb79f4debf7773b144efec518d307d15d992bbd6..dc907d0b01cca27ad360583531a34d19722f60ee 100644 (file)
--- a/test/details.js
+++ b/test/details.js
@@ -1,4 +1,6 @@
-const assert = require('assert');
+import assert from 'assert';
+
+const reverse_only = !!process.env.REVERSE_ONLY;
 
 describe('Details Page', function () {
   let page;
@@ -22,7 +24,7 @@ describe('Details Page', function () {
     before(async function () {
       page = await browser.newPage();
       await page.goto('http://localhost:9999/details.html');
-      await page.type('input[type=edit]', 'n3');
+      await page.type('input[type=edit]', 'N6');
       await page.click('button[type=submit]');
       await page.waitForSelector('#api-request');
     });
@@ -59,45 +61,43 @@ describe('Details Page', function () {
     });
 
     it('should have OSM link', async function () {
+      const url = 'https://www.openstreetmap.org/relation/1155956';
 
-      assert.strictEqual((await page.$$('a[href="https://www.openstreetmap.org/relation/1155956"]')).length, 2);
+      assert.strictEqual((await page.$$eval(`a[href="${url}"]`, (links) => links.length)), 2);
     });
 
-    it('should change page url and add new header on clicking display keywords', async function () {
-      let current_url;
-      let display_headers;
-      let [display_keywords_btn] = await page.$x("//a[contains(text(), 'display keywords')]");
+    it('should have a link to postcode which includes country code', async function () {
+      const url = 'search.html?postalcode=9490&country=li';
 
-      await display_keywords_btn.click();
-      await page.waitForNavigation();
-
-      current_url = new URL(await page.url());
-      assert.strictEqual(current_url.searchParams.get('keywords'), '1');
+      assert.strictEqual((await page.$$eval(`a[href="${url}"]`, (links) => links.length)), 1);
+    });
 
-      await page.waitForSelector('h3');
-      display_headers = await page.$$eval('h3', elements => elements.map(el => el.textContent));
-      assert.deepStrictEqual(display_headers, ['Name Keywords', 'Address Keywords']);
+    // Reverse-only installation have no search index, therefore no keywords
+    if (!reverse_only) {
+      it('should change url and add new header on clicking display keywords', async function () {
+        let current_url;
+        let display_headers;
+        let [display_keywords_btn] = await page.$$(
+          "xpath/.//a[contains(text(), 'display keywords')]"
+        );
 
-      let page_content = await page.$eval('body', el => el.textContent);
-      assert.ok(page_content.includes('qwaansshe')); // one of the name keywords
-    });
+        await display_keywords_btn.evaluate(node => node.click());
+        await page.waitForNavigation();
 
-    it('should change page url on clicking display child places', async function () {
-      let current_url;
-      let [child_places_btn] = await page.$x("//a[contains(text(), 'display child places')]");
+        current_url = new URL(await page.url());
+        assert.strictEqual(current_url.searchParams.get('keywords'), '1');
 
-      await child_places_btn.click();
-      await page.waitForNavigation();
-      await page.waitForSelector('table#address');
+        await page.waitForSelector('h3');
+        display_headers = await page.$$eval('h3', elements => elements.map(el => el.textContent));
+        assert.deepStrictEqual(display_headers, ['Name Keywords', 'Address Keywords']);
 
-      current_url = new URL(await page.url());
-      assert.strictEqual(current_url.searchParams.get('hierarchy'), '1');
+        let page_content = await page.$eval('body', el => el.textContent);
+        assert.ok(page_content.includes('vadouz')); // one of the name keywords
+      });
+    }
 
-      let page_content = await page.$eval('body', el => el.textContent);
-      assert.ok(page_content.includes('Alte Landstrasse')); // one of the streets
-    });
 
-    it('should support case-insenstive search, can navigate to new page', async function () {
+    it('should support case-insensitive search, can navigate to new page', async function () {
       let input_field = await page.$('input[type=edit]');
       await input_field.click({ clickCount: 3 });
       await input_field.type('w375257537');
@@ -108,6 +108,38 @@ describe('Details Page', function () {
     });
   });
 
+  describe('With street search - a place that is parent of buildings', function () {
+    before(async function () {
+      page = await browser.newPage();
+      await page.goto('http://localhost:9999/details.html?osmtype=W&osmid=32703083');
+      await page.waitForSelector('.container .row');
+    });
+
+    after(async function () {
+      await page.close();
+    });
+
+    it('should change page url on clicking display child places', async function () {
+      let page_content = await page.$eval('body', el => el.textContent);
+      assert.ok(page_content.includes('Gafleistrasse'));
+
+      let current_url;
+      let [child_places_btn] = await page.$$(
+        "xpath/.//a[contains(text(), 'display child places')]"
+      );
+
+      await child_places_btn.evaluate(node => node.click());
+      await page.waitForNavigation();
+      await page.waitForSelector('table#address');
+
+      current_url = new URL(await page.url());
+      assert.strictEqual(current_url.searchParams.get('hierarchy'), '1');
+
+      page_content = await page.$eval('body', el => el.textContent);
+      assert.ok(page_content.includes('bus_stop')); // parent of several bus stops
+    });
+  });
+
   describe('Place without name, keywords, hierarchy', function () {
     // e.g. a numeric house number
     before(async function () {
@@ -124,7 +156,9 @@ describe('Details Page', function () {
       let page_content = await page.$eval('body', el => el.textContent);
 
       assert.ok(page_content.includes('Name No Name'));
-      assert.ok(page_content.includes('Place has no keywords'));
+      if (!process.env.REVERSE_ONLY) {
+        assert.ok(page_content.includes('Place has no keywords'));
+      }
       assert.ok(page_content.includes('Place is not parent of other places'));
     });
   });