More tests

Author: Damir

src/find-path.js

@@ -12,13 +12,7 @@ const folderSeparator = '.';
  * @param  {Object} options [posthtml options]
  * @return {String|boolean}
  */
-function findPathFromTagName(node, options) {
-  if (!node.attrs) {
-    node.attrs = {};
-  }
-
-  const {tag} = node;
-
+function findPathFromTagName({tag}, options) {
   // Get module filename from tag name
   //  remove prefix "x-"
   //  replace dot "." with slash "/"
@@ -29,8 +23,8 @@ function findPathFromTagName(node, options) {
     .join(path.sep)
     .concat(folderSeparator, options.fileExtension);
 
-  // Find module by defined namespace in options.namespaces
-  //  or by defined roots in options.roots
+  // Find module by defined namespace in options.namespaces when tag has '::'
+  //  otherwise by defined roots in options.roots
   return tag.includes(options.namespaceSeparator) ?
     findPathByNamespace(tag, fileNameFromTag.split(options.namespaceSeparator), options) :
     findPathByRoot(tag, fileNameFromTag, options);

test/templates/components/form/index.html

@@ -0,0 +1 @@
+<form>My Form</form>

test/templates/custom/dark/components/label/index.html

@@ -0,0 +1 @@
+<label>My Label</label>

test/templates/dark/components/label/index.html

@@ -0,0 +1 @@
+<label class="bg-dark text-light">My Dark Label</label>

test/test-errors.js

@@ -8,11 +8,44 @@ const clean = html => html.replace(/(\n|\t)/g, '').trim();
 test('Must fail when namespace path is not found without fallback root', async t => {
   const actual = `<div><x-empty-namespace::button>Submit</x-empty-namespace::button></div>`;
 
-  await t.throwsAsync(async () => posthtml([plugin({root: './test/templates', namespaces: [{name: 'empty-namespace', root: './test/templates/empty-namespace'}]})]).process(actual).then(result => clean(result.html)));
+  await t.throwsAsync(async () => posthtml([plugin({root: './test/templates', namespaceFallback: false, namespaces: [{name: 'empty-namespace', root: './test/templates/empty-namespace'}]})]).process(actual).then(result => clean(result.html)));
 });
 
 test('Must fail when namespace path is not found with fallback root', async t => {
   const actual = `<div><x-empty-namespace::button>Submit</x-empty-namespace::button></div>`;
 
-  await t.throwsAsync(async () => posthtml([plugin({root: './test/templates', namespaces: [{name: 'empty-namespace', root: './test/templates/empty-namespace'}]})]).process(actual).then(result => clean(result.html)));
+  await t.throwsAsync(async () => posthtml([plugin({root: './test/templates', namespaceFallback: true, namespaces: [{name: 'empty-namespace', root: './test/templates/empty-namespace'}]})]).process(actual).then(result => clean(result.html)));
+});
+
+test('Must fail when namespace is unknown', async t => {
+  const actual = `<div><x-unknown-namespace::button>Submit</x-unknown-namespace::button></div>`;
+
+  await t.throwsAsync(async () => posthtml([plugin({root: './test/templates'})]).process(actual).then(result => clean(result.html)));
+});
+
+test('Must return node as-is when namespace is unknown with strict mode disabled', async t => {
+  const actual = `<div><x-unknown-namespace::button>Submit</x-unknown-namespace::button></div>`;
+  const expected = `<div><x-unknown-namespace::button>Submit</x-unknown-namespace::button></div>`;
+
+  const html = await posthtml([plugin({root: './test/templates', strict: false})]).process(actual).then(result => clean(result.html));
+
+  t.is(html, expected);
+});
+
+test('Must return node as-is when namespace is empty with strict mode disabled', async t => {
+  const actual = `<div><x-empty-namespace::button>Submit</x-empty-namespace::button></div>`;
+  const expected = `<div><x-empty-namespace::button>Submit</x-empty-namespace::button></div>`;
+
+  const html = await posthtml([plugin({root: './test/templates', strict: false, namespaces: [{name: 'empty-namespace', root: './test/templates/empty-namespace'}]})]).process(actual).then(result => clean(result.html));
+
+  t.is(html, expected);
+});
+
+test('Must return node as-is when all defined roots are empty with strict mode disabled', async t => {
+  const actual = `<div><x-button>Submit</x-button></div>`;
+  const expected = `<div><x-button>Submit</x-button></div>`;
+
+  const html = await posthtml([plugin({root: './test/templates/empty-root', strict: false})]).process(actual).then(result => clean(result.html));
+
+  t.is(html, expected);
 });

test/test-x-tag.js

@@ -32,6 +32,15 @@ test('Must process component with namespace to html', async t => {
   t.is(html, expected);
 });
 
+test('Must process component with namespace to html using index file', async t => {
+  const actual = `<x-dark::label></x-dark::label>`;
+  const expected = `<label class="bg-dark text-light">My Dark Label</label>`;
+
+  const html = await posthtml([plugin({root: './test/templates', namespaces: [{name: 'dark', root: './test/templates/dark/components'}]})]).process(actual).then(result => clean(result.html));
+
+  t.is(html, expected);
+});
+
 test(`Must process component with namespace's fallback path`, async t => {
   const actual = `<x-dark::modal></x-dark::modal>`;
   const expected = `<div>Modal</div>`;
@@ -41,6 +50,15 @@ test(`Must process component with namespace's fallback path`, async t => {
   t.is(html, expected);
 });
 
+test(`Must process component with namespace's fallback path using index file`, async t => {
+  const actual = `<x-dark::form></x-dark::form>`;
+  const expected = `<form>My Form</form>`;
+
+  const html = await posthtml([plugin({root: './test/templates', namespaces: [{name: 'dark', root: './test/templates/dark/components', fallback: './test/templates/components'}]})]).process(actual).then(result => clean(result.html));
+
+  t.is(html, expected);
+});
+
 test(`Must process component with namespace's custom path`, async t => {
   const actual = `<x-dark::button><slot name="content">My button</slot></x-dark::button>`;
   const expected = `<button class="bg-dark-custom text-light-custom">My button</button>`;
@@ -49,3 +67,12 @@ test(`Must process component with namespace's custom path`, async t => {
 
   t.is(html, expected);
 });
+
+test(`Must process component with namespace's custom path using index file`, async t => {
+  const actual = `<x-dark::label></x-dark::label>`;
+  const expected = `<label>My Label</label>`;
+
+  const html = await posthtml([plugin({root: './test/templates', namespaces: [{name: 'dark', root: './test/templates/dark/components', custom: './test/templates/custom/dark/components'}]})]).process(actual).then(result => clean(result.html));
+
+  t.is(html, expected);
+});