From 750ea9d153295b69131c5390c179cc677c5044c4 Mon Sep 17 00:00:00 2001
From: JonnyWong16 <9099342+JonnyWong16@users.noreply.github.com>
Date: Sun, 3 Jan 2021 13:50:20 -0800
Subject: [PATCH] Fix server dashboard tests

---
 tests/payloads.py    |  7 +++++++
 tests/test_server.py | 25 ++++++++++++++-----------
 2 files changed, 21 insertions(+), 11 deletions(-)

diff --git a/tests/payloads.py b/tests/payloads.py
index 940e0d57..dc54dc42 100644
--- a/tests/payloads.py
+++ b/tests/payloads.py
@@ -22,3 +22,10 @@ SONOS_RESOURCES = """<MediaContainer size="3">
   <Player title="Speaker 3" machineIdentifier="RINCON_12345678901234563:1234567893" deviceClass="speaker" product="Sonos" platform="Sonos" platformVersion="56.0-76060" protocol="plex" protocolVersion="1" protocolCapabilities="timeline,playback,playqueues,provider-playback" lanIP="192.168.1.13"/>
 </MediaContainer>
 """
+
+SERVER_RESOURCES = """<MediaContainer size="3">
+<StatisticsResources timespan="6" at="1609708609" hostCpuUtilization="0.000" processCpuUtilization="0.207" hostMemoryUtilization="64.946" processMemoryUtilization="3.665"/>
+<StatisticsResources timespan="6" at="1609708614" hostCpuUtilization="5.000" processCpuUtilization="0.713" hostMemoryUtilization="64.939" processMemoryUtilization="3.666"/>
+<StatisticsResources timespan="6" at="1609708619" hostCpuUtilization="10.000" processCpuUtilization="4.415" hostMemoryUtilization="64.281" processMemoryUtilization="3.669"/>
+</MediaContainer>
+"""
diff --git a/tests/test_server.py b/tests/test_server.py
index 24ffa3c4..7331e3fe 100644
--- a/tests/test_server.py
+++ b/tests/test_server.py
@@ -11,6 +11,7 @@ from plexapi.utils import download
 from requests import Session
 
 from . import conftest as utils
+from .payloads import SERVER_RESOURCES
 
 
 def test_server_attr(plex, account):
@@ -345,14 +346,14 @@ def test_server_allowMediaDeletion(account):
 def test_server_system_accounts(plex):
     accounts = plex.systemAccounts()
     assert len(accounts)
-    account = accounts[0]
+    account = accounts[-1]
     assert utils.is_bool(account.autoSelectAudio)
     assert account.defaultAudioLanguage == "en"
     assert account.defaultSubtitleLanguage == "en"
     assert utils.is_int(account.id, gte=0)
     assert len(account.key)
-    assert account.name == ""
-    assert account.subtitleMode == 1
+    assert len(account.name)
+    assert account.subtitleMode == 0
     assert account.thumb == ""
     assert account.accountID == account.id
     assert account.accountKey == account.key
@@ -361,12 +362,12 @@ def test_server_system_accounts(plex):
 def test_server_system_devices(plex):
     devices = plex.systemDevices()
     assert len(devices)
-    device = devices[0]
+    device = devices[-1]
     assert utils.is_datetime(device.createdAt)
     assert utils.is_int(device.id)
     assert len(device.key)
-    assert len(device.name)
-    assert len(device.platform)
+    assert len(device.name) or device.name == ""
+    assert len(device.platform) or device.platform == ""
     
 
 @pytest.mark.authenticated
@@ -414,13 +415,15 @@ def test_server_dashboard_bandwidth_filters(plex):
 
 
 @pytest.mark.authenticated
-def test_server_dashboard_resources(plex):
+def test_server_dashboard_resources(plex, requests_mock):
+    url = plex.url("/statistics/resources")
+    requests_mock.get(url, text=SERVER_RESOURCES)
     resourceData = plex.resources()
     assert len(resourceData)
     resource = resourceData[0]
     assert utils.is_datetime(resource.at)
-    assert utils.is_float(resource.hostCpuUtilization)
-    assert utils.is_float(resource.hostMemoryUtilization)
-    assert utils.is_float(resource.processCpuUtilization)
-    assert utils.is_float(resource.processMemoryUtilization)
+    assert utils.is_float(resource.hostCpuUtilization, gte=0.0)
+    assert utils.is_float(resource.hostMemoryUtilization, gte=0.0)
+    assert utils.is_float(resource.processCpuUtilization, gte=0.0)
+    assert utils.is_float(resource.processMemoryUtilization, gte=0.0)
     assert resource.timespan == 6  # Default seconds timespan