Update experimenter api environment variable (as part of Android PR)

.env

@@ -2,16 +2,7 @@
 
 # Base URL for the experimenter API (defaults to the production instance)
 #
-EXPERIMENTER_API_PREFIX="https://experimenter.services.mozilla.com/api/v7/"
-
-# API calls with parameters to fetch experiments we want to display.
-# https://htmlpreview.github.io/?https://github.com/mozilla/experimenter/blob/main/docs/experimenter/swagger-ui.html has more info.
-#
-# Live experiments
-EXPERIMENTER_API_CALL_LIVE="experiments/?status=Live&application=firefox-desktop"
-
-# Completed experiments
-EXPERIMENTER_API_CALL_COMPLETED="experiments/?status=Complete&application=firefox-desktop"
+EXPERIMENTER_API_PREFIX="https://experimenter.services.mozilla.com/api/v7/experiments/"
 
 # Looker configurables
 IS_LOOKER_ENABLED=false

.env.sample

@@ -1,7 +1,8 @@
 # Copy this to .env.local to set local variables
 
-# Preview
-# EXPERIMENTER_API_CALL="experiments/?status=Preview&application=firefox-desktop"
+# Base URL for the experimenter API (defaults to the production instance)
+#
+EXPERIMENTER_API_PREFIX="https://experimenter.services.mozilla.com/api/v7/experiments/"
 
 # Disable Auth0 for dev && preview environments
 IS_AUTH_ENABLED='false'

.github/copilot-instructions.md

@@ -0,0 +1,25 @@
+# GitHub Copilot Instructions
+
+## Case-Sensitive Filesystem
+
+Some of our development happens on a case-sensitive filesystem. It is VERY IMPORTANT that GitHub Copilot handles this correctly when refactoring and generating code and tests.
+
+### Guidelines
+
+1. **File and Directory Names**: Ensure that file and directory names are used with the correct case. For example, `MyFile.ts` and `myfile.ts` are different files on a case-sensitive filesystem.
+2. **Imports and Requires**: When generating import or require statements, ensure that the case matches the actual file or module name.
+3. **Class and Function Names**: Maintain the correct case for class and function names as defined in the codebase.
+4. **Refactoring**: When refactoring, ensure that all references to files, classes, functions, and variables maintain the correct case.
+
+### Specific Instructions for Component Files
+
+When working with component files where the component name is uppercase and the file name contains lowercase, ensure the following:
+
+1. **Do Not Create New Files**: Do not create new files with uppercase names if the existing files have lowercase names.
+2. **Correct File Names**: Use the existing files with the correct case.
+3. **Correct Imports**: When importing components in other files, ensure the import statement uses the correct case:
+   ```tsx
+   import Component from "@/app/component";
+   ```
+
+By following these guidelines, we can avoid issues related to case sensitivity and unnecessary file creation in our development process.

.github/copilot-test-generation.md

@@ -0,0 +1,25 @@
+# GitHub Copilot Instructions
+
+## Case-Sensitive Filesystem
+
+Some of our development happens on a case-sensitive filesystem. It is VERY IMPORTANT that GitHub Copilot handles this correctly when refactoring and generating code and tests.
+
+### Guidelines
+
+1. **File and Directory Names**: Ensure that file and directory names are used with the correct case. For example, `MyFile.ts` and `myfile.ts` are different files on a case-sensitive filesystem.
+2. **Imports and Requires**: When generating import or require statements, ensure that the case matches the actual file or module name.
+3. **Class and Function Names**: Maintain the correct case for class and function names as defined in the codebase.
+4. **Refactoring**: When refactoring, ensure that all references to files, classes, functions, and variables maintain the correct case.
+
+### Specific Instructions for Component Files
+
+When working with component files where the component name is uppercase and the file name contains lowercase, ensure the following:
+
+1. **Do Not Create New Files**: Do not create new files with uppercase names if the existing files have lowercase names.
+2. **Correct File Names**: Use the existing files with the correct case.
+3. **Correct Imports**: When importing components in other files, ensure the import statement uses the correct case:
+   ```tsx
+   import Component from "@/app/component";
+   ```
+
+By following these guidelines, we can avoid issues related to case sensitivity and unnecessary file creation in our development process.

CHANGELOG.md

@@ -1,3 +1,9 @@
+## Tueday, April 15th, 2025
+
+### Updated
+
+- Updated the environment variables to use an `EXPERIMENTER_API_PREFIX`. Instead of having separate environment variables for live and completed experiments, we are now using this prefix and adding on required parameters in the code when necessary.
+
 ## Friday, April 11th, 2025
 
 ### Fixed

CONTRIBUTING.md

@@ -9,8 +9,8 @@ practices solidify.
 
    `cp .env.sample .env.local`
 
-1. Modify variables in .env.local, e.g. uncommenting the `EXPERIMENTER_API_CALL`
-   line will switch from the default of live experiments to preview experiments
+1. Modify variables in .env.local
+   1. The `EXPERIMENTER_API_PREFIX` environment variable is the base URL for the experimenter API and it defaults to the production instance. Paramaters like `status` and `application` are added to this base URL in the code to specify live/completed status and different platforms (ie. fenix, ios. firefox-desktop).
 
 ## Running the development server
 

README.md

@@ -8,8 +8,8 @@ A bunch of the code, ideas, and links come from the OMC team's work week Hackath
 
    `cp .env.sample .env.local`
 
-1. Modify variables in .env.local, e.g. uncommenting the `EXPERIMENTER_API_CALL`
-   line will switch from the default of live experiments to preview experiments
+1. Modify variables in .env.local
+   1. The `EXPERIMENTER_API_PREFIX` environment variable is the base URL for the experimenter API and it defaults to the production instance. Paramaters like `status` and `application` are added to this base URL in the code to specify live/completed status and different platforms (ie. fenix, ios. firefox-desktop).
 
 ## Running the development server
 

__tests__/app/dashboard.test.tsx

@@ -11,9 +11,17 @@ global.fetch = jest.fn(() =>
   }),
 ) as jest.Mock;
 
+const mockFetchData = {
+  localData: [],
+  experimentAndBranchInfo: [],
+  totalExperiments: 0,
+  msgRolloutInfo: [],
+  totalRolloutExperiments: 0,
+};
+
 describe.skip("Dashboard", () => {
   it("all timeline pill ids exist in the Dashboard component in /", async () => {
-    const dashboard = await render(await (<Dashboard />));
+    const dashboard = await render(await (<Dashboard {...mockFetchData} />));
 
     const firefox = dashboard.getByTestId("firefox");
     const experiments = dashboard.getByTestId("live_experiments");

__tests__/lib/nimbusRecipeCollection.test.ts

@@ -2,6 +2,9 @@ import { NimbusRecipe } from "@/lib/nimbusRecipe";
 import { NimbusRecipeCollection } from "@/lib/nimbusRecipeCollection";
 import { ExperimentFakes } from "@/__tests__/ExperimentFakes.mjs";
 import { RecipeInfo } from "@/app/columns";
+import { Platform } from "@/lib/types";
+
+const platform: Platform = "firefox-desktop";
 
 const fakeFetchData = [ExperimentFakes.recipe()];
 global.fetch = jest.fn(() =>
@@ -27,6 +30,29 @@ describe("NimbusRecipeCollection", () => {
 
       expect(recipes).toEqual([new NimbusRecipe(fakeFetchData[0])]);
     });
+
+    it("constructs the correct URL for live experiments", async () => {
+      const nimbusRecipeCollection = new NimbusRecipeCollection(
+        false,
+        platform,
+      ); //XXX YYY
+      await nimbusRecipeCollection.fetchRecipes();
+
+      expect(global.fetch).toHaveBeenCalledWith(
+        `${process.env.EXPERIMENTER_API_PREFIX}?status=Live&application=${platform}`,
+        { credentials: "omit" },
+      );
+    });
+
+    it("constructs the correct URL for completed experiments", async () => {
+      const nimbusRecipeCollection = new NimbusRecipeCollection(true);
+      await nimbusRecipeCollection.fetchRecipes();
+
+      expect(global.fetch).toHaveBeenCalledWith(
+        `${process.env.EXPERIMENTER_API_PREFIX}?status=Complete&application=${platform}`,
+        { credentials: "omit" },
+      );
+    });
   });
 
   describe("getExperimentAndBranchInfos", () => {

app/android/page.tsx

@@ -1,5 +1,25 @@
 import { Dashboard } from "@/app/dashboard";
+import { fetchData } from "@/app/fetchData";
+import { Platform } from "@/lib/types";
 
-export default function Page() {
-  return <Dashboard platform={"android"} />;
+const platform: Platform = "fenix";
+
+export default async function Page() {
+  const {
+    localData,
+    experimentAndBranchInfo,
+    totalExperiments,
+    msgRolloutInfo,
+    totalRolloutExperiments,
+  } = await fetchData(platform);
+
+  return (
+    <Dashboard
+      platform={platform}
+      experimentAndBranchInfo={experimentAndBranchInfo}
+      totalExperiments={totalExperiments}
+      msgRolloutInfo={msgRolloutInfo}
+      totalRolloutExperiments={totalRolloutExperiments}
+    />
+  );
 }