This page is not created by, affiliated with, or supported by Slack Technologies, Inc.
2019-08-08
Channels
- # announcements (43)
- # architecture (4)
- # beginners (115)
- # calva (6)
- # cider (4)
- # circleci (4)
- # clara (3)
- # clj-kondo (6)
- # cljs-dev (10)
- # cljsrn (1)
- # clojars (1)
- # clojure (108)
- # clojure-boston (1)
- # clojure-dev (4)
- # clojure-europe (5)
- # clojure-italy (6)
- # clojure-nl (13)
- # clojure-uk (17)
- # clojurescript (47)
- # core-async (2)
- # cursive (13)
- # data-science (1)
- # datavis (15)
- # datomic (12)
- # graphql (5)
- # juxt (10)
- # kaocha (20)
- # lumo (1)
- # off-topic (27)
- # pedestal (4)
- # reitit (2)
- # shadow-cljs (115)
- # spacemacs (4)
- # sql (74)
- # tools-deps (79)
- # vim (15)
- # xtdb (2)
Howdy! First things first, thank you for the work on clj-kondo, it's found a few genuine bugs for us, really useful bit of software.
I think I've found a false-positive with the :missing-test-assertion
linter in 2019.07.31-alpha
(ns false-positive.reproduction-test
(:require [clojure.test :refer (are deftest)]))
(deftest missing-test-assertion-false-positive
(are [v expected?] (expected? (contains? #{:bar :baz} v))
:foo false?
:bar true?
:baz true?))
Results in the warning /Users/gordon/Development/foo/test/false_positive/reproduction_test.clj::: warning: missing test assertion
I haven't reproduced this in a clean environment outside my employer's main repo, but we don't have any :missing-test-assertion
config in .clj-kondo/config.edn
so I hope it's generally valid.
If not I'll happily put the legwork in to get a properly standalone test case.