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/5] 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 b66f2dc45..703496754 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 df0a7eb55..7cb4ba70f 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 00e73bae2..a8c3655b7 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 4ecca497c..d34f35f67 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 18e15c74e..7e157ae43 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/5] 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 7cb4ba70f..89874b408 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/5] 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 000000000..e46b98590
--- /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 a8c3655b7..cd9c53fe2 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/5] 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 e46b98590..9f9e560ff 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 ebd044ce8a17162b16f2592f78540eaf11c307d6 Mon Sep 17 00:00:00 2001
From: Github Build Bot <buildbot@fishtownanalytics.com>
Date: Thu, 29 Feb 2024 20:54:27 +0000
Subject: [PATCH 5/5] Delete changie files that were moved to dbt-core for
 release

---
 .changes/unreleased/Docs-20230615-105157.yaml | 6 ------
 1 file changed, 6 deletions(-)
 delete mode 100644 .changes/unreleased/Docs-20230615-105157.yaml

diff --git a/.changes/unreleased/Docs-20230615-105157.yaml b/.changes/unreleased/Docs-20230615-105157.yaml
deleted file mode 100644
index 327e33b2e..000000000
--- a/.changes/unreleased/Docs-20230615-105157.yaml
+++ /dev/null
@@ -1,6 +0,0 @@
-kind: Docs
-body: Add analytics for dbt.com
-time: 2023-06-15T10:51:57.838991+02:00
-custom:
-  Author: b-per
-  Issue: "430"