Browse Source

update isOS method

ell 1 year ago
parent
commit
71f379e268
3 changed files with 8 additions and 11 deletions
  1. 6 9
      testing/classes/TestMethod.lua
  2. 1 1
      testing/classes/TestSuite.lua
  3. 1 1
      testing/tests/graphics.lua

+ 6 - 9
testing/classes/TestMethod.lua

@@ -376,17 +376,14 @@ TestMethod = {
 
   -- @method - TestMethod:isOS()
   -- @desc - checks for a specific OS (or list of OSs)
-  -- @param {string||table} - single OS string or list of OSs that are allowed, 
-  --          these will be checked agaisnt love.system.getOS()'s return value
+  -- @param {string/s} - each arg passed will be checked as a valid OS, as long
+  --                     as one passed the function will return true
   -- @return {boolean} - returns true if one of the OSs given matches actual OS
-  isOS = function(self, oss)
-    if type(oss) == 'table' then
-      for o=1,#oss do
-        if oss[o] == love.test.current_os then return true end
-      end
-    else
-      return love.test.current_os == oss
+  isOS = function(self, ...)
+    for os=1,select("#", ...) do
+      if select(os, ...) == love.test.current_os then return true end
     end
+    return false
   end,
 
   -- @method - TestMethod:evaluateTest()

+ 1 - 1
testing/classes/TestSuite.lua

@@ -23,7 +23,7 @@ TestSuite = {
       delayed = nil,
       fakequit = false,
       windowmode = true,
-      current_os = love.system.getOS(),
+      current_os = love._os,
 
       -- love modules to test
       audio = {},

+ 1 - 1
testing/tests/graphics.lua

@@ -1467,7 +1467,7 @@ love.test.graphics.captureScreenshot = function(test)
   TextCommand = prevtextcommand
   test:assertNotNil(cbdata)
 
-  if test:isOS({'iOS', 'Android'}) then
+  if test:isOS('iOS', 'Android') then
     -- Mobile operating systems don't let us control the window resolution,
     -- so we can't compare the reference image properly.
     test:assertTrue(true, 'skip test')