Merge pull request #978 from gabrielburnworth/os_update_button

Minor fixes and additions

webpack/devices/components/fbos_settings/__tests__/fbos_details_test.tsx

@@ -17,9 +17,8 @@ describe("<FbosDetails/>", () => {
     return {
       botInfoSettings: bot.hardware.informational_settings,
       dispatch: jest.fn(x => x(jest.fn(), fakeState)),
-      sourceFbosConfig: (x) => {
-        return { value: bot.hardware.configuration[x], consistent: true };
-      }
+      sourceFbosConfig: x =>
+        ({ value: bot.hardware.configuration[x], consistent: true }),
     };
   };
 
@@ -33,6 +32,12 @@ describe("<FbosDetails/>", () => {
     p.botInfoSettings.firmware_commit = "fakeFwCommit";
     p.botInfoSettings.soc_temp = 48.3;
     p.botInfoSettings.wifi_level = -49;
+    // tslint:disable-next-line:no-any
+    (p.botInfoSettings as any).uptime = 0;
+    // tslint:disable-next-line:no-any
+    (p.botInfoSettings as any).memory_usage = 0;
+    // tslint:disable-next-line:no-any
+    (p.botInfoSettings as any).disk_usage = 0;
 
     const wrapper = mount(<FbosDetails {...p} />);
     ["Environment", "fakeEnv",
@@ -44,6 +49,9 @@ describe("<FbosDetails/>", () => {
       "FAKETARGET CPU temperature", "48.3", "C",
       "WiFi Strength", "-49dBm",
       "Beta release Opt-In",
+      "Uptime", "0s",
+      "Memory usage", "0MB",
+      "Disk usage", "0%",
     ]
       .map(string => expect(wrapper.text()).toContain(string));
   });
@@ -94,6 +102,19 @@ describe("<FbosDetails/>", () => {
     expect(wrapper.text()).toContain("CPU temperature: unknown");
     expect(wrapper.text()).not.toContain("&deg;C");
   });
+
+  it("doesn't display extra metrics when bot is offline", () => {
+    const p = fakeProps();
+    // tslint:disable-next-line:no-any
+    (p.botInfoSettings as any).uptime = undefined;
+    // tslint:disable-next-line:no-any
+    (p.botInfoSettings as any).memory_usage = undefined;
+    // tslint:disable-next-line:no-any
+    (p.botInfoSettings as any).disk_usage = undefined;
+    const wrapper = mount(<FbosDetails {...p} />);
+    ["uptime", "usage"].map(metric =>
+      expect(wrapper.text().toLowerCase()).not.toContain(metric));
+  });
 });
 
 describe("colorFromTemp()", () => {