Clojurians Log v2

Thanks @seancorfield for helping to figure this out!

Is your own code using Slingshot perhaps, and bringing in an older version than Kaocha needs?

Slingshot? Sorry... what that should be?

You're probably going to need to share your deps.edn file and you may do better getting help directly in the #kaocha channel (I don't use Kaocha so I'm speculating based on that stacktrace and what the kaocha.version-check namespace contains).

This is how my deps.end looks like

{:deps    {org.clojure/clojure {:mvn/version "1.10.1"}

           ;; DATABASE ACCESS
           org.clojure/java.jdbc {:mvn/version "0.6.1"}

           org.clojure/data.json {:mvn/version "2.1.1"}
           javax.servlet/servlet-api {:mvn/version "2.5"}
           org.flatland/ordered {:mvn/version "1.15.10"}

           ;; LOGGING
           com.taoensso/timbre {:mvn/version "4.10.0"}
           org.slf4j/slf4j-api {:mvn/version "1.7.14"}
           com.fzakaria/slf4j-timbre {:mvn/version "0.3.13"}
           org.slf4j/log4j-over-slf4j {:mvn/version "1.7.14"}
           org.slf4j/jul-to-slf4j {:mvn/version "1.7.14"}
           org.slf4j/jcl-over-slf4j {:mvn/version "1.7.14"}

           }

 :paths   ["src" "classes" "resources"]

 :aliases {:test
           {:extra-paths ["test"]
            :extra-deps {lambdaisland/kaocha {:mvn/version "1.87.1366"}
                         com.h2database/h2 {:mvn/version  "1.3.148"}
                         org.postgresql/postgresql {:mvn/version "42.2.20"}
                         expectations/expectations {:mvn/version "2.2.0-rc3"}
                         org.eclipse.jetty/jetty-servlet-tester {:mvn/version "7.0.0.M2"}}
            :main-opts ["-m" "kaocha.runner" #_"--watch" #_"--no-capture-output" "--fail-fast"]}
		   }
 }

I don't see anything wrong with that (tried it locally -- only Kaocha brings in Slingshot and it brings in the latest version). What's in your Kaocha config file?

I have none.

And you're running this with clojure -M:test ?

kind of... I've follow the install instruction on kaocha site and have created a bin/kaocha with this:

#!/usr/bin/env bash

clojure -M:test "$@"

When I run that, using your deps.edn, I get warnings:

WARNING: Did not load a configuration file and using the defaults.
This is fine for experimenting, but for long-term use, we recommend creating a configuration file to avoid changes in behavior between releases.
To create a configuration file using the current defaults and configuration file location, create a file named tests.edn that contains '#kaocha/v1 {}'.

sorry... I have a tests.edn indeed.

And what does clojure -version print out?

#kaocha/v1
 {:plugins [:hooks]
  :kaocha.hooks/pre-load [docx.hooks/pre-load-hook]}

clojure -version Clojure CLI version 1.11.1.1347

OK, good. And docx.hooks/pre-load-hook is in your own code?

yes, it is

(ns docx.hooks
  (:require [ :as io]))

(defn pre-load-hook [test-plan]
  (compile 'docx.core)
  (compile 'docx.servlet)
  test-plan)

^ it's this simple

And that's under your test tree?

yes:

OK, so run this command to get an interactive REPL:

clj -Sdeps '{:aliases {:repl {:main-opts ["-r"]}}}' -M:test:repl

(require 'docx.hooks)
(docx.hooks/pre-load-hook nil)

(that alias/repl trick is a way to override the :main-opts in your :test alias so you can get a REPL with test stuff loaded)

It gives an error:

OK, so you probably have an error in one of your files, and because it's a compile error when Kaocha tries to load your code, the stacktrace is misleading.

So start a fresh REPL like you did above and try to (require 'docx.core) and then (require 'docx.servlet) to see if that helps narrow down where your syntax error is.

(unless all your code is up on GH or somewhere I can just go look at it and see if I can spot the problem)

It may be easier to use your editor and just try loading each file into the REPL and see what errors you get...

what i can't understand is that this very base code can compile and generate a .jar file that can be run with success...

😉 it was a comment (;) inside the (ns ... )

thank you very much for the tips

Happy to help! Glad you figured it out 🙂

Remember that your test code does not get compiled or loaded when you make a JAR -- I assume the ns error was in a test file somewhere?

it was both in both places

😞 i was wrong

it didn't work

it's really weird, @seancorfield

the first time I do a require, it dont work, but if I call require again, it works.

That's why I said you needed a fresh REPL for the test...

So if docx.core won't load, start a new REPL and try to require each of the nses it requires in its ns. Repeat recursively until you find the culprit.

The "Syntax error .. at (.. 11)" suggests it's something in an ns form somewhere in your code.

(if this was up on GitHub, I could take a look and probably find it faster)

I'm cutting piece by piece to find out...

to running test, do I need do first compile the classes?

i mean... I do need to add a 'compile' step before calling the tests?

I'm getting errors like that: clj -X:test Running tests in #{"test"} Syntax error (ClassNotFoundException) compiling at (docx/servlet_test.clj:20:20). docx.servlet

(i've dumped the kaocha and now using just the regular clojure test runner... then I'm getting those error about having not compile the classes)

Do what I suggested above: start a REPL from the command-line and just try to require a namespace.

You need to do little tiny steps in the REPL.

Until you've figured out what is causing the syntax error, there's no point in trying to run the tests at all.

from repl it's working

there's no syntax error

37 minutes ago, you said there was still.

https://clojurians.slack.com/archives/C08LK2DH7/p1700173723295269?thread_ts=1700166725.087079&cid=C08LK2DH7

Fresh REPL. One require. It failed.

yeah... but that i could figure out

and fixed

the only thing now is this code:

it's a test

and it need the class docx.servlet

but it can't find this class

Why did you switch away from Kaocha and the pre load hook?

so I'm thinking that I will need to compile that in some way

That hook was what was compiling the classes you need.

because I couldn't find out any useful messages when using it

with the clojure test runner, there's some "hook" alternative that I can use do compile the classes?

No.

If you think you've "fixed" the problem in your code, go back and repeat this experiment and make sure you can load and run the hook: https://clojurians.slack.com/archives/C08LK2DH7/p1700169047926159?thread_ts=1700166725.087079&cid=C08LK2DH7

The problem was with the sfl4j shit... after I've removed (comment out) that error on spec alfa gone

Now I came back to Kaocha

and is almost 100% working.. there's just this error:

that has to be with some logging shit..

Finnaly! Everything working... in the end, I think it was just an 'require' that wasn't present... thank you very much, @seancorfield You're the best Sean that I know of.

Logging is a giant pain. Personally, I think Timbre makes it worse. We use clojure.tools.logging and log4j2 at work (after trying a lot of other things).

Systematic, step-by-step debugging FTW 🙂