Test multiple versions of Python

CHANGELOG.md

@@ -15,11 +15,56 @@ The types of changes are:
 * `Fixed` for any bug fixes.
 * `Security` in case of vulnerabilities.
 
-## [Unreleased](https://github.com/ethyca/fides/compare/1.8.2...main)
+## [Unreleased](https://github.com/ethyca/fides/compare/1.8.4...main)
 
 ### Added
 
-* Added more taxonomy fields that can be edited via the UI [#1000](https://github.com/ethyca/fides/pull/1000)
+* Dataset generation enhancements using Fides Classify for Plus users:
+  * Added toggle for enabling classify during generation. [#1057](https://github.com/ethyca/fides/pull/1057)
+  * Initial implementation of API request to kick off classify, with confirmation modal. [#1069](https://github.com/ethyca/fides/pull/1069)
+  * Initial Classification & Review status for generated datasets. [#1074](https://github.com/ethyca/fides/pull/1074)
+* New page to add a system via yaml [#1062](https://github.com/ethyca/fides/pull/1062)
+  * Skeleton of page to add a system manually [#1068](https://github.com/ethyca/fides/pull/1068)
+
+### Changed
+
+* Changed behavior of `load_default_taxonomy` to append instead of upsert [#1040](https://github.com/ethyca/fides/pull/1040)
+
+## [1.8.4](https://github.com/ethyca/fides/compare/1.8.3...1.8.4) - 2022-09-09
+
+### Added
+
+* Initial system management page [#1054](https://github.com/ethyca/fides/pull/1054)
+
+### Changed
+
+* Deleting a taxonomy field with children will now cascade delete all of its children as well. [#1042](https://github.com/ethyca/fides/pull/1042)
+
+### Fixed
+
+* Fixed navigating directly to frontend routes loading index page instead of the correct static page for the route.
+* Fix truncated evaluation error messages [#1053](https://github.com/ethyca/fides/pull/1053)
+
+## [1.8.3](https://github.com/ethyca/fides/compare/1.8.2...1.8.3) - 2022-09-06
+
+### Added
+
+* Added more taxonomy fields that can be edited via the UI [#1000](https://github.com/ethyca/fides/pull/1000) [#1028](https://github.com/ethyca/fides/pull/1028)
+* Added the ability to add taxonomy fields via the UI [#1019](https://github.com/ethyca/fides/pull/1019)
+* Added the ability to delete taxonomy fields via the UI [#1006](https://github.com/ethyca/fides/pull/1006)
+  * Only non-default taxonomy entities can be deleted [#1023](https://github.com/ethyca/fides/pull/1023)
+* Prevent deleting taxonomy `is_default` fields and from adding `is_default=True` fields via the API [#990](https://github.com/ethyca/fides/pull/990).
+* Added a "Custom" tag to distinguish user defined taxonomy fields from default taxonomy fields in the UI [#1027](https://github.com/ethyca/fides/pull/1027)
+* Added initial support for enabling Fides Plus [#1037](https://github.com/ethyca/fides/pull/1037)
+  * The `useFeatures` hook can be used to check if `plus` is enabled.
+  * Navigating to/from the Data Map page is gated behind this feature.
+  * Plus endpoints are served from the private Plus image.
+
+### Fixed
+
+* Fixed failing mypy tests [#1030](https://github.com/ethyca/fides/pull/1030)
+* Fixed an issue where `fides push --diff` would return a false positive diff [#1026](https://github.com/ethyca/fides/pull/1026)
+* Pinned pydantic version to < 1.10.0 to fix an error in finding referenced fides keys [#1045](https://github.com/ethyca/fides/pull/1045)
 
 ## [1.8.2](https://github.com/ethyca/fides/compare/1.8.1...1.8.2) - 2022-08-18
 

Dockerfile

@@ -67,6 +67,8 @@ RUN : \
 ## Python Dependencies ##
 #########################
 
+# Install multiple versions of Python
+
 COPY dev-requirements.txt dev-requirements.txt
 RUN pip install -r dev-requirements.txt
 

clients/ctl/admin-ui/__tests__/features/common/zones/config.test.ts

@@ -0,0 +1,103 @@
+import { configureZones, resolveZone } from "~/features/common/zones";
+
+describe("resolveZone", () => {
+  // Allow mocking and resetting the node env in this test suite.
+  const env = process.env as { NODE_ENV: string };
+  afterEach(() => {
+    env.NODE_ENV = "test";
+  });
+
+  const config = configureZones({
+    development: {
+      host: "localhost:3000",
+    },
+
+    zones: [
+      {
+        basePath: "/datamap",
+        development: {
+          host: "localhost:4000",
+        },
+      },
+      {
+        basePath: "/no/dev",
+      },
+    ],
+  });
+
+  describe("Within the root zone", () => {
+    const basePath = "/";
+
+    it("Does not modify a link within the root zone", () => {
+      env.NODE_ENV = "development";
+
+      const link = resolveZone({ config, basePath, href: "/root/route" });
+      expect(link).toMatchObject({
+        href: "/root/route",
+        basePath: "/",
+      });
+    });
+
+    it("Links to the dev host of another zone", () => {
+      env.NODE_ENV = "development";
+
+      const link = resolveZone({ config, basePath, href: "/datamap" });
+      expect(link).toMatchObject({
+        href: "http://localhost:4000/datamap",
+        basePath: "/datamap",
+      });
+    });
+
+    it("Does not link to the dev host in production", () => {
+      env.NODE_ENV = "production";
+
+      const link = resolveZone({ config, basePath, href: "/datamap" });
+      expect(link).toMatchObject({
+        href: "/datamap",
+        basePath: "/datamap",
+      });
+    });
+
+    it("Does not link to the dev host if none is configured", () => {
+      env.NODE_ENV = "development";
+
+      const link = resolveZone({ config, basePath, href: "/no/dev" });
+      expect(link).toMatchObject({
+        href: "/no/dev",
+        basePath: "/no/dev",
+      });
+    });
+  });
+
+  describe("Within a sub zone", () => {
+    const basePath = "/datamap";
+
+    it("Strips the base path of the sub-zone", () => {
+      const link = resolveZone({ config, basePath, href: "/datamap/route" });
+      expect(link).toMatchObject({
+        href: "/route",
+        basePath: "/datamap",
+      });
+    });
+
+    it("Links to the dev host of the root zone", () => {
+      env.NODE_ENV = "development";
+
+      const link = resolveZone({ config, basePath, href: "/root/route" });
+      expect(link).toMatchObject({
+        href: "http://localhost:3000/root/route",
+        basePath: "/",
+      });
+    });
+
+    it("Does not link to the dev host in production", () => {
+      env.NODE_ENV = "production";
+
+      const link = resolveZone({ config, basePath, href: "/root/route" });
+      expect(link).toMatchObject({
+        href: "/root/route",
+        basePath: "/",
+      });
+    });
+  });
+});

clients/ctl/admin-ui/__tests__/features/taxonomy-helpers.test.tsx

@@ -0,0 +1,18 @@
+import { parentKeyFromFidesKey } from "~/features/taxonomy/helpers";
+
+describe("taxonomy helpers", () => {
+  describe("parent key from fides key", () => {
+    it("should handle nested fides keys", () => {
+      const fidesKey = "grandparent.parent.child";
+      expect(parentKeyFromFidesKey(fidesKey)).toEqual("grandparent.parent");
+    });
+    it("should handle empty fides key", () => {
+      const fidesKey = "";
+      expect(parentKeyFromFidesKey(fidesKey)).toEqual("");
+    });
+    it("should handle single fides key", () => {
+      const fidesKey = "root";
+      expect(parentKeyFromFidesKey(fidesKey)).toEqual("");
+    });
+  });
+});

clients/ctl/admin-ui/__tests__/features/taxonomy-transform.test.tsx

@@ -14,16 +14,19 @@ describe("data category transform", () => {
                 description: "Account's city level address data.",
                 label: "Account City",
                 value: "account.contact.city",
+                is_default: true,
               },
             ],
             description: "Contact data related to a system account.",
             label: "Account Contact Data",
             value: "account.contact",
+            is_default: true,
           },
         ],
         description: "Data related to a system account.",
         label: "Account Data",
         value: "account",
+        is_default: true,
       },
       {
         children: [
@@ -32,11 +35,13 @@ describe("data category transform", () => {
             description: "Data used to manage access to the system.",
             label: "Authentication Data",
             value: "system.authentication",
+            is_default: true,
           },
         ],
         description: "Data unique to, and under control of the system.",
         label: "System Data",
         value: "system",
+        is_default: true,
       },
       {
         children: [
@@ -46,6 +51,7 @@ describe("data category transform", () => {
               "Data derived from user provided data or as a result of user actions in the system.",
             label: "Derived Data",
             value: "user.derived",
+            is_default: true,
           },
           {
             children: [
@@ -55,6 +61,7 @@ describe("data category transform", () => {
                   "Data provided or created directly by a user that is not identifiable.",
                 label: "User Provided Non-Identifiable Data",
                 value: "user.provided.nonidentifiable",
+                is_default: true,
               },
               {
                 children: [
@@ -63,31 +70,36 @@ describe("data category transform", () => {
                     description: "Age range data.",
                     label: "User Provided Non-Specific Age",
                     value: "user.provided.identifiable.non_specific_age",
+                    is_default: true,
                   },
                   {
                     children: [],
                     description:
                       "Data related to the individual's political opinions.",
                     label: "Political Opinion",
                     value: "user.provided.identifiable.political_opinion",
+                    is_default: true,
                   },
                 ],
                 description:
                   "Data provided or created directly by a user that is linked to or identifies a user.",
                 label: "User Provided Identifiable Data",
                 value: "user.provided.identifiable",
+                is_default: true,
               },
             ],
             description:
               "Data provided or created directly by a user of the system.",
             label: "User Provided Data",
             value: "user.provided",
+            is_default: true,
           },
         ],
         description:
           "Data related to the user of the system, either provided directly or derived based on their usage.",
         label: "User Data",
         value: "user",
+        is_default: true,
       },
     ]);
   });
@@ -100,20 +112,23 @@ describe("data category transform", () => {
         value: "anonymous_user",
         description:
           "An individual that is unidentifiable to the systems. Note - This should only be applied to truly anonymous users where there is no risk of re-identification",
+        is_default: true,
         children: [],
       },
       {
         label: "Citizen Voter",
         value: "citizen_voter",
         description:
           "An individual registered to voter with a state or authority.",
+        is_default: true,
         children: [],
       },
       {
         label: "Commuter",
         value: "commuter",
         description:
           "An individual that is traveling or transiting in the context of location tracking.",
+        is_default: true,
         children: [],
       },
     ]);