Rename flags + rename targets + add disallowed section to feature mapping

src/__test__/providerTests.ts

@@ -15,40 +15,52 @@ export function makeProviderMocks() {
 
 const sourceA = {
   id: 'a',
+  name: 'A',
   rank: 1,
   disabled: false,
+  flags: [],
 } as Sourcerer;
 const sourceB = {
   id: 'b',
+  name: 'B',
   rank: 2,
   disabled: false,
+  flags: [],
 } as Sourcerer;
 const sourceCDisabled = {
   id: 'c',
+  name: 'C',
   rank: 3,
   disabled: true,
+  flags: [],
 } as Sourcerer;
 const sourceAHigherRank = {
   id: 'a',
+  name: 'A',
   rank: 100,
   disabled: false,
+  flags: [],
 } as Sourcerer;
 const sourceGSameRankAsA = {
   id: 'g',
+  name: 'G',
   rank: 1,
   disabled: false,
+  flags: [],
 } as Sourcerer;
 const fullSourceYMovie = {
   id: 'y',
   name: 'Y',
   rank: 105,
   scrapeMovie: vi.fn(),
+  flags: [],
 } as Sourcerer;
 const fullSourceYShow = {
   id: 'y',
   name: 'Y',
   rank: 105,
   scrapeShow: vi.fn(),
+  flags: [],
 } as Sourcerer;
 const fullSourceZBoth = {
   id: 'z',
@@ -56,6 +68,7 @@ const fullSourceZBoth = {
   rank: 106,
   scrapeMovie: vi.fn(),
   scrapeShow: vi.fn(),
+  flags: [],
 } as Sourcerer;
 
 const embedD = {

src/__test__/providers/checks.test.ts

@@ -1,12 +1,14 @@
 import { mockEmbeds, mockSources } from '@/__test__/providerTests';
+import { FeatureMap } from '@/main/targets.ts';
 import { getProviders } from '@/providers/get';
 import { vi, describe, it, expect, afterEach } from 'vitest';
 
 const mocks = await vi.hoisted(async () => (await import('../providerTests.ts')).makeProviderMocks());
 vi.mock('@/providers/all', () => mocks);
 
-const features = {
+const features: FeatureMap = {
   requires: [],
+  disallowed: []
 }
 
 describe('getProviders()', () => {

src/__test__/utils/features.test.ts

@@ -0,0 +1,77 @@
+import { FeatureMap, Flags, flags, flagsAllowedInFeatures } from "@/main/targets";
+import { describe, it, expect } from "vitest";
+
+describe('flagsAllowedInFeatures()', () => {
+  function checkFeatures(featureMap: FeatureMap, flags: Flags[], output: boolean) {
+    expect(flagsAllowedInFeatures(featureMap, flags)).toEqual(output);
+  }
+
+  it('should check required correctly', () => {
+    checkFeatures({
+      requires: [],
+      disallowed: []
+    }, [], true);
+    checkFeatures({
+      requires: [flags.CORS_ALLOWED],
+      disallowed: []
+    }, [flags.CORS_ALLOWED], true);
+    checkFeatures({
+      requires: [flags.CORS_ALLOWED],
+      disallowed: []
+    }, [], false);
+    checkFeatures({
+      requires: [flags.CORS_ALLOWED, flags.IP_LOCKED],
+      disallowed: []
+    }, [flags.CORS_ALLOWED, flags.IP_LOCKED], true);
+    checkFeatures({
+      requires: [flags.IP_LOCKED],
+      disallowed: []
+    }, [flags.CORS_ALLOWED], false);
+    checkFeatures({
+      requires: [flags.IP_LOCKED],
+      disallowed: []
+    }, [], false);
+  });
+  
+  it('should check disallowed correctly', () => {
+    checkFeatures({
+      requires: [],
+      disallowed: []
+    }, [], true);
+    checkFeatures({
+      requires: [],
+      disallowed: [flags.CORS_ALLOWED]
+    }, [], true);
+    checkFeatures({
+      requires: [],
+      disallowed: [flags.CORS_ALLOWED]
+    }, [flags.CORS_ALLOWED], false);
+    checkFeatures({
+      requires: [],
+      disallowed: [flags.CORS_ALLOWED]
+    }, [flags.IP_LOCKED], true);
+    checkFeatures({
+      requires: [],
+      disallowed: [flags.CORS_ALLOWED, flags.IP_LOCKED]
+    }, [flags.CORS_ALLOWED], false);
+  });
+  
+  it('should pass mixed tests', () => {
+    checkFeatures({
+      requires: [flags.CORS_ALLOWED],
+      disallowed: [flags.IP_LOCKED]
+    }, [], false);
+    checkFeatures({
+      requires: [flags.CORS_ALLOWED],
+      disallowed: [flags.IP_LOCKED]
+    }, [flags.CORS_ALLOWED], true);
+    checkFeatures({
+      requires: [flags.CORS_ALLOWED],
+      disallowed: [flags.IP_LOCKED]
+    }, [flags.IP_LOCKED], false);
+    checkFeatures({
+      requires: [flags.CORS_ALLOWED],
+      disallowed: [flags.IP_LOCKED]
+    }, [flags.IP_LOCKED, flags.CORS_ALLOWED], false);
+  });
+});