Update "removed" definition

We'll say that an item is only removed if it (1) has an "in_source"
annotation and it is false. This annotation should always be present
for metrics, but is not currently set for any other object.

src/components/ItemList.svelte

@@ -10,7 +10,7 @@
   import Label from "./Label.svelte";
 
   import { filterUncollectedItems } from "../state/filter";
-  import { isExpired } from "../state/metrics";
+  import { isExpired, isRemoved } from "../state/items";
   import { pageState, updateURLState } from "../state/stores";
 
   let DEFAULT_ITEMS_PER_PAGE = 20;
@@ -189,7 +189,7 @@
                       />
                     {/each}
                   {/if}
-                  {#if !item.in_source}
+                  {#if isRemoved(item)}
                     <Label text="removed" />
                   {:else if isExpired(item.expires)}
                     <Label text="expired" />

src/pages/MetricDetail.svelte

@@ -20,7 +20,7 @@
   import { pageTitle, pageState, updateURLState } from "../state/stores";
   import { getBigQueryURL } from "../state/urls";
 
-  import { isExpired } from "../state/metrics";
+  import { isExpired } from "../state/items";
 
   export let params;
 

src/state/filter.js

@@ -1,4 +1,4 @@
-import { isExpired } from "./metrics";
+import { isExpired } from "./items";
 
 export const filterUncollectedItems = (items, showUncollected) =>
   items.filter(

src/state/metrics.js → src/state/items.js

@@ -1,3 +1,9 @@
+export const isRemoved = (item) => {
+  // only consider an item expired if it has an in_source property
+  // and it is false
+  return item.in_source === false;
+};
+
 export const isExpired = (expiryDate) => {
   if (expiryDate === "never" || expiryDate === undefined) {
     return false;

tests/state.items.test.js

@@ -0,0 +1,25 @@
+import { isExpired, isRemoved } from "../src/state/items";
+
+describe("checking if a date has expired", () => {
+  it("returns True if input is 'never' or undefined", () => {
+    expect(isExpired("never") || isExpired(undefined)).toEqual(false);
+  });
+  it("returns True if input is a date from the past", () => {
+    expect(isExpired("2021-01-01")).toEqual(true);
+  });
+  it("returns False if input is a date from the future", () => {
+    expect(isExpired("3021-01-01")).toEqual(false);
+  });
+});
+
+describe("isRemoved works as expected", () => {
+  it("returns true if in_source is false", () => {
+    expect(isRemoved({ in_source: false })).toEqual(true);
+  });
+  it("returns false if in_source is undefined", () => {
+    expect(isRemoved({})).toEqual(false);
+  });
+  it("returns false if in_source is true", () => {
+    expect(isRemoved({ in_source: true })).toEqual(false);
+  });
+});