From 4c47caa3ec4a9097b6ffbcf5c443c140056c8923 Mon Sep 17 00:00:00 2001
From: Gerda Shank <gerda@dbtlabs.com>
Date: Tue, 27 Feb 2024 17:13:56 -0500
Subject: [PATCH 1/9] Attempt to update to handle unit tests

---
 src/app/components/column_details/column_details.html | 4 ++--
 src/app/components/references/index.js                | 4 +++-
 src/app/index.routes.js                               | 9 +++++++++
 src/app/services/node_selection_service.js            | 5 +++--
 src/app/services/project_service.js                   | 6 ++++++
 5 files changed, 23 insertions(+), 5 deletions(-)

diff --git a/src/app/components/column_details/column_details.html b/src/app/components/column_details/column_details.html
index b66f2dc4..70349675 100644
--- a/src/app/components/column_details/column_details.html
+++ b/src/app/components/column_details/column_details.html
@@ -13,7 +13,7 @@
                         <th style="background-color: white; position: sticky; top: 0; z-index: 1;">Type</th>
                         <th style="background-color: white; position: sticky; top: 0; z-index: 1;">Description</th>
                         <th style="width: 1px; background-color: white; position: sticky; top: 0; z-index: 1;">Constraints</th>
-                        <th style="width: 1px; background-color: white; position: sticky; top: 0; z-index: 1;">Tests</th>
+                        <th style="width: 1px; background-color: white; position: sticky; top: 0; z-index: 1;">Data Tests</th>
                         <th style="width: 1px; background-color: white; position: sticky; top: 0; z-index: 1;" class='text-center'>More?</th>
                     </tr>
                 </thead>
@@ -104,7 +104,7 @@ <h5>Constraints</h5>
                                 </div>
 
                                 <div ng-show="column.tests && column.tests.length" style="margin-bottom: 15px">
-                                    <h5>Generic Tests</h5>
+                                    <h5>Generic Data Tests</h5>
                                     <ul class="list-unstyled" style="margin-top: 2px">
                                         <li ng-repeat="test in column.tests" ng-switch on="test.short" class='text-light'>
                                             <span ng-switch-when="F">
diff --git a/src/app/components/references/index.js b/src/app/components/references/index.js
index df0a7eb5..7cb4ba70 100644
--- a/src/app/components/references/index.js
+++ b/src/app/components/references/index.js
@@ -34,7 +34,9 @@ angular
                 } else if (type == 'seed') {
                     return 'Seeds';
                 } else if (type == 'test') {
-                    return 'Tests';
+                    return 'Data Tests';
+                } else if (type == 'unit_test') {
+                    return "Unit Tests';
                 } else if (type == 'snapshot') {
                     return 'Snapshots'
                 } else if (type == 'analysis') {
diff --git a/src/app/index.routes.js b/src/app/index.routes.js
index 00e73bae..a8c3655b 100644
--- a/src/app/index.routes.js
+++ b/src/app/index.routes.js
@@ -14,6 +14,7 @@ const templates = {
     snapshot: require('./docs/snapshot.html'),
     seed: require('./docs/seed.html'),
     test: require('./docs/test.html'),
+    unit_test: require('./docs/test.html'),
     analysis: require('./docs/analysis.html'),
     macro: require('./docs/macro.html'),
     exposure: require('./docs/exposure.html'),
@@ -86,6 +87,14 @@ angular
                 unique_id: {type: 'string'}
             },
         })
+        .state('dbt.unit_test', {
+            url: 'unit_test/:unique_id?section&' + graph_params,
+            controller: 'TestCtrl',
+            templateUrl: templates.test,
+            params: {
+                unique_id: {type: 'string'}
+            },
+        })
         .state('dbt.analysis', {
             url: 'analysis/:unique_id?section&' + graph_params,
             controller: 'AnalysisCtrl',
diff --git a/src/app/services/node_selection_service.js b/src/app/services/node_selection_service.js
index 4ecca497..d34f35f6 100644
--- a/src/app/services/node_selection_service.js
+++ b/src/app/services/node_selection_service.js
@@ -18,6 +18,7 @@ angular
             'snapshot',
             'source',
             'test',
+            'unit_test',
             'analysis',
             'exposure',
             'metric',
@@ -37,7 +38,7 @@ angular
         options: {
             packages: [],
             tags: [null],
-            resource_types: ['model', 'seed', 'snapshot', 'source', 'test', 'analysis', 'exposure', 'metric', 'semantic_model'],
+            resource_types: ['model', 'seed', 'snapshot', 'source', 'test', 'analysis', 'exposure', 'metric', 'semantic_model', 'unit_test'],
         }
     };
 
@@ -63,7 +64,7 @@ angular
             include_selection = '+metric:' + node.name;
         } else if (node && node.resource_type == 'semantic_model') {
             include_selection = '+semantic_model:' + node.name;
-        } else if (node && _.includes(['analysis', 'test'], node.resource_type)) {
+        } else if (node && _.includes(['analysis', 'test', 'unit_test'], node.resource_type)) {
             include_selection = '+' + node.name;
         } else {
             include_selection = "";
diff --git a/src/app/services/project_service.js b/src/app/services/project_service.js
index 18e15c74..7e157ae4 100644
--- a/src/app/services/project_service.js
+++ b/src/app/services/project_service.js
@@ -185,6 +185,12 @@ angular
                 node.label = node.name;
             });
 
+            // Add unit tests into nodes
+            _.each(service.files.manifest.unit_tests, function(node) {
+                service.files.manifest.nodes[node.unique_id] = node;
+                node.label = node.name;
+            });
+
             var adapter = service.files.manifest.metadata.adapter_type;
             var macros = clean_project_macros(service.files.manifest.macros, adapter);
             service.files.manifest.macros = macros;

From fc5423868d9663d00fd04b2c10f94f489cbfc484 Mon Sep 17 00:00:00 2001
From: Gerda Shank <gerda@dbtlabs.com>
Date: Wed, 28 Feb 2024 17:10:21 -0500
Subject: [PATCH 2/9] Fix unmatched quote

---
 src/app/components/references/index.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/app/components/references/index.js b/src/app/components/references/index.js
index 7cb4ba70..89874b40 100644
--- a/src/app/components/references/index.js
+++ b/src/app/components/references/index.js
@@ -36,7 +36,7 @@ angular
                 } else if (type == 'test') {
                     return 'Data Tests';
                 } else if (type == 'unit_test') {
-                    return "Unit Tests';
+                    return 'Unit Tests';
                 } else if (type == 'snapshot') {
                     return 'Snapshots'
                 } else if (type == 'analysis') {

From 51b4f5fc51ebad7f04082372ae59478521e455de Mon Sep 17 00:00:00 2001
From: Gerda Shank <gerda@dbtlabs.com>
Date: Thu, 29 Feb 2024 15:02:56 -0500
Subject: [PATCH 3/9] add unit_test.html

---
 src/app/docs/unit_test.html | 61 +++++++++++++++++++++++++++++++++++++
 src/app/index.routes.js     |  2 +-
 2 files changed, 62 insertions(+), 1 deletion(-)
 create mode 100644 src/app/docs/unit_test.html

diff --git a/src/app/docs/unit_test.html b/src/app/docs/unit_test.html
new file mode 100644
index 00000000..e46b9859
--- /dev/null
+++ b/src/app/docs/unit_test.html
@@ -0,0 +1,61 @@
+<style>
+/* TODO */
+.section-target {
+    top: -8em;
+}
+
+.noflex {
+    flex: 0 0 160px !important;
+}
+
+.highlight {
+    color: #24292e;
+    background-color: white;
+}
+
+</style>
+
+<div class='app-scroll'>
+    <div class="app-links app-sticky">
+        <div class="app-title">
+            <div class="app-frame app-pad app-flush-bottom">
+                <h1>
+                    <span class="break">{{ model.name }}</span>
+                    <small>test</small>
+                </h1>
+            </div>
+        </div>
+        <div class="app-frame app-pad-h">
+            <ul class="nav nav-tabs">
+                <li ui-sref-active='active'><a ui-sref="dbt.test({'#': 'description'})">Description</a></li>
+                <li ui-sref-active='active' ng-show = "parentsLength != 0"><a ui-sref="dbt.test({'#': 'depends_on'})">Depends On</a></li>
+                <li ui-sref-active='active'><a ui-sref="dbt.test({'#': 'code'})">SQL</a></li>
+            </ul>
+        </div>
+    </div>
+    <div class="app-details">
+        <div class="app-frame app-pad">
+            <section class="section">
+                <div class="section-target" id="description"></div>
+                <div class="section-content">
+                    <h6>Description</h6>
+                    <div class="panel">
+                        <div class="panel-body">
+                            <div ng-if="model.description" class="model-markdown" marked="model.description"></div>
+                            <div ng-if="!model.description">This {{ model.resource_type }} is not currently documented</div>
+                        </div>
+                    </div>
+                </div>
+            </section>
+
+            <section class="section" ng-show = "parentsLength != 0">
+                <div class="section-target" id="depends_on"></div>
+                <div class="section-content">
+                    <h6>Depends On</h6>
+                    <reference-list references="parents" node="model" />
+                </div>
+            </section>
+
+        </div>
+    </div>
+</div>
diff --git a/src/app/index.routes.js b/src/app/index.routes.js
index a8c3655b..cd9c53fe 100644
--- a/src/app/index.routes.js
+++ b/src/app/index.routes.js
@@ -14,7 +14,7 @@ const templates = {
     snapshot: require('./docs/snapshot.html'),
     seed: require('./docs/seed.html'),
     test: require('./docs/test.html'),
-    unit_test: require('./docs/test.html'),
+    unit_test: require('./docs/unit_test.html'),
     analysis: require('./docs/analysis.html'),
     macro: require('./docs/macro.html'),
     exposure: require('./docs/exposure.html'),

From 801255b70605a79b4b6ab1a2414b5ad9cbc85a67 Mon Sep 17 00:00:00 2001
From: Gerda Shank <gerda@dbtlabs.com>
Date: Thu, 29 Feb 2024 15:51:50 -0500
Subject: [PATCH 4/9] Tweak unit test page

---
 src/app/docs/unit_test.html | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/src/app/docs/unit_test.html b/src/app/docs/unit_test.html
index e46b9859..9f9e560f 100644
--- a/src/app/docs/unit_test.html
+++ b/src/app/docs/unit_test.html
@@ -21,15 +21,14 @@
             <div class="app-frame app-pad app-flush-bottom">
                 <h1>
                     <span class="break">{{ model.name }}</span>
-                    <small>test</small>
+                    <small>unit test</small>
                 </h1>
             </div>
         </div>
         <div class="app-frame app-pad-h">
             <ul class="nav nav-tabs">
-                <li ui-sref-active='active'><a ui-sref="dbt.test({'#': 'description'})">Description</a></li>
-                <li ui-sref-active='active' ng-show = "parentsLength != 0"><a ui-sref="dbt.test({'#': 'depends_on'})">Depends On</a></li>
-                <li ui-sref-active='active'><a ui-sref="dbt.test({'#': 'code'})">SQL</a></li>
+                <li ui-sref-active='active'><a ui-sref="dbt.unit_test({'#': 'description'})">Description</a></li>
+                <li ui-sref-active='active' ng-show = "parentsLength != 0"><a ui-sref="dbt.unit_test({'#': 'depends_on'})">Depends On</a></li>
             </ul>
         </div>
     </div>

From a0007169d8b3360959db7b00e097a3186fe93fd5 Mon Sep 17 00:00:00 2001
From: Gerda Shank <gerda@dbtlabs.com>
Date: Thu, 29 Feb 2024 16:03:42 -0500
Subject: [PATCH 5/9] Add "Unit Test" to some headings for diagnostic purposes

---
 src/app/docs/unit_test.html | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/app/docs/unit_test.html b/src/app/docs/unit_test.html
index 9f9e560f..bbeb730f 100644
--- a/src/app/docs/unit_test.html
+++ b/src/app/docs/unit_test.html
@@ -27,8 +27,8 @@ <h1>
         </div>
         <div class="app-frame app-pad-h">
             <ul class="nav nav-tabs">
-                <li ui-sref-active='active'><a ui-sref="dbt.unit_test({'#': 'description'})">Description</a></li>
-                <li ui-sref-active='active' ng-show = "parentsLength != 0"><a ui-sref="dbt.unit_test({'#': 'depends_on'})">Depends On</a></li>
+                <li ui-sref-active='active'><a ui-sref="dbt.unit_test({'#': 'description'})">Description of Unit Test</a></li>
+                <li ui-sref-active='active' ng-show = "parentsLength != 0"><a ui-sref="dbt.unit_test({'#': 'depends_on'})">Unit Test Depends On</a></li>
             </ul>
         </div>
     </div>
@@ -37,7 +37,7 @@ <h1>
             <section class="section">
                 <div class="section-target" id="description"></div>
                 <div class="section-content">
-                    <h6>Description</h6>
+                    <h6>Description of Unit Test</h6>
                     <div class="panel">
                         <div class="panel-body">
                             <div ng-if="model.description" class="model-markdown" marked="model.description"></div>
@@ -50,7 +50,7 @@ <h6>Description</h6>
             <section class="section" ng-show = "parentsLength != 0">
                 <div class="section-target" id="depends_on"></div>
                 <div class="section-content">
-                    <h6>Depends On</h6>
+                    <h6>Unit Test Depends On</h6>
                     <reference-list references="parents" node="model" />
                 </div>
             </section>

From ad81775a236c979ff7a7f61a3ff6fc89ad7c44dc Mon Sep 17 00:00:00 2001
From: Gerda Shank <gerda@dbtlabs.com>
Date: Fri, 1 Mar 2024 14:39:33 -0500
Subject: [PATCH 6/9] Reverse order of test and unit_test

---
 src/app/index.routes.js | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/app/index.routes.js b/src/app/index.routes.js
index cd9c53fe..621e6c80 100644
--- a/src/app/index.routes.js
+++ b/src/app/index.routes.js
@@ -13,8 +13,8 @@ const templates = {
     source: require('./docs/source.html'),
     snapshot: require('./docs/snapshot.html'),
     seed: require('./docs/seed.html'),
-    test: require('./docs/test.html'),
     unit_test: require('./docs/unit_test.html'),
+    test: require('./docs/test.html'),
     analysis: require('./docs/analysis.html'),
     macro: require('./docs/macro.html'),
     exposure: require('./docs/exposure.html'),
@@ -79,16 +79,16 @@ angular
                 unique_id: {type: 'string'}
             },
         })
-        .state('dbt.test', {
-            url: 'test/:unique_id?section&' + graph_params,
+        .state('dbt.unit_test', {
+            url: 'unit_test/:unique_id?section&' + graph_params,
             controller: 'TestCtrl',
             templateUrl: templates.test,
             params: {
                 unique_id: {type: 'string'}
             },
         })
-        .state('dbt.unit_test', {
-            url: 'unit_test/:unique_id?section&' + graph_params,
+        .state('dbt.test', {
+            url: 'test/:unique_id?section&' + graph_params,
             controller: 'TestCtrl',
             templateUrl: templates.test,
             params: {

From 064913479b0a2e062d0c0b8115c5c89815ca6c67 Mon Sep 17 00:00:00 2001
From: Gerda Shank <gerda@dbtlabs.com>
Date: Fri, 1 Mar 2024 15:02:55 -0500
Subject: [PATCH 7/9] specify unit_test template

---
 src/app/index.routes.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/app/index.routes.js b/src/app/index.routes.js
index 621e6c80..3986e2e2 100644
--- a/src/app/index.routes.js
+++ b/src/app/index.routes.js
@@ -82,7 +82,7 @@ angular
         .state('dbt.unit_test', {
             url: 'unit_test/:unique_id?section&' + graph_params,
             controller: 'TestCtrl',
-            templateUrl: templates.test,
+            templateUrl: templates.unit_test,
             params: {
                 unique_id: {type: 'string'}
             },

From 00554b2cabc7da8cc8f8b595f0ab9cd1489801f0 Mon Sep 17 00:00:00 2001
From: Gerda Shank <gerda@dbtlabs.com>
Date: Thu, 7 Mar 2024 13:48:50 -0500
Subject: [PATCH 8/9] Remove extraneous "Unit Test" from unit_test template

---
 src/app/docs/unit_test.html | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/app/docs/unit_test.html b/src/app/docs/unit_test.html
index bbeb730f..9f9e560f 100644
--- a/src/app/docs/unit_test.html
+++ b/src/app/docs/unit_test.html
@@ -27,8 +27,8 @@ <h1>
         </div>
         <div class="app-frame app-pad-h">
             <ul class="nav nav-tabs">
-                <li ui-sref-active='active'><a ui-sref="dbt.unit_test({'#': 'description'})">Description of Unit Test</a></li>
-                <li ui-sref-active='active' ng-show = "parentsLength != 0"><a ui-sref="dbt.unit_test({'#': 'depends_on'})">Unit Test Depends On</a></li>
+                <li ui-sref-active='active'><a ui-sref="dbt.unit_test({'#': 'description'})">Description</a></li>
+                <li ui-sref-active='active' ng-show = "parentsLength != 0"><a ui-sref="dbt.unit_test({'#': 'depends_on'})">Depends On</a></li>
             </ul>
         </div>
     </div>
@@ -37,7 +37,7 @@ <h1>
             <section class="section">
                 <div class="section-target" id="description"></div>
                 <div class="section-content">
-                    <h6>Description of Unit Test</h6>
+                    <h6>Description</h6>
                     <div class="panel">
                         <div class="panel-body">
                             <div ng-if="model.description" class="model-markdown" marked="model.description"></div>
@@ -50,7 +50,7 @@ <h6>Description of Unit Test</h6>
             <section class="section" ng-show = "parentsLength != 0">
                 <div class="section-target" id="depends_on"></div>
                 <div class="section-content">
-                    <h6>Unit Test Depends On</h6>
+                    <h6>Depends On</h6>
                     <reference-list references="parents" node="model" />
                 </div>
             </section>

From df3eb3fd435e9a26c86f02ec232e8bd19a7371d3 Mon Sep 17 00:00:00 2001
From: Gerda Shank <gerda@dbtlabs.com>
Date: Mon, 11 Mar 2024 14:03:51 -0400
Subject: [PATCH 9/9] Changie

---
 .changes/unreleased/Docs-20240311-140344.yaml | 6 ++++++
 1 file changed, 6 insertions(+)
 create mode 100644 .changes/unreleased/Docs-20240311-140344.yaml

diff --git a/.changes/unreleased/Docs-20240311-140344.yaml b/.changes/unreleased/Docs-20240311-140344.yaml
new file mode 100644
index 00000000..6a78c95e
--- /dev/null
+++ b/.changes/unreleased/Docs-20240311-140344.yaml
@@ -0,0 +1,6 @@
+kind: Docs
+body: Enable display of unit tests
+time: 2024-03-11T14:03:44.490834-04:00
+custom:
+  Author: gshank
+  Issue: "501"