saml20-clj icon indicating copy to clipboard operation
saml20-clj copied to clipboard

`saml20-clj.sp.response-test/assertions-test` is actually failing

Open escherize opened this issue 2 years ago • 0 comments

This line: https://github.com/metabase/saml20-clj/blob/master/test/saml20_clj/sp/response_test.clj#L22

Should actually read:

(doseq [{:keys [response], :as response-map} (test/responses)
          :when (not (test/invalid-confirmation-data? response-map))] ;;<-- call the function
)

Functions are truthy, and (not ) is false, so these tests were never run, eek. Because of this, this doseq didn't execute any bodies aka tests weren't being run. I tweaked it locally and there are 2 failing tests there.

escherize avatar Feb 14 '24 20:02 escherize