Requirement check closure is called too many times

[zenled]

As explained here (1. Requirement check closure is called multiple times), the requirements check closure should be called only one time for each configuration.

Example:

{
   "ios":[
      {
         "required_version":"1.2.4",
         "last_version_available":"1.8.0",
         "notify_last_version_frequency":"ALWAYS",
         "requirements":{
            "region":"hr",
            "bluetooth":"5.0"
         }
      }
   ]
}

let url = URL(string: "https://pastebin.com/raw/qtUwSvgR")!

let options = PoVRequestOptions()

options.addRequirement(key: "region"){(value) -> Bool in
   return true // breakpoint A
}

options.addRequirement(key: "bluetooth"){(value) -> Bool in
   return true // breakpoint B
}

PrinceOfVersions.checkForUpdates(from: url, options: options){ response in
    switch response.result {
    case .success(let updateResultData):
        print("Update version: \(updateResultData.updateVersion)")
    case .failure:
        print("Failed")
        break
    }
}

Since there is only one configuration, I would expect the breakpoints A and B to be triggered exactly once. In my testing both A and B are triggered twice.

One more thing. The order does not seem to be consistent between runs (even multiple runs on the same build). Sometimes it is A,B,A,B and sometimes B,A,B,A. While this library does not promise a consistent order, this does seem strange and might indicate of a hidden bug.

[Filip2Stojanovski]

Hi zenled,
Thank you for reaching out and bringing this to our attention. Apologies for the delay in responding. Our team will review this issue promptly.
We appreciate your patience and understanding. If you have any additional details that could help, please feel free to share them here.
Best regards,
Filip