emacs-bug-tracker
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[debbugs-tracker] bug#28250: closed (slf4j-api: enable tests)


From: GNU bug Tracking System
Subject: [debbugs-tracker] bug#28250: closed (slf4j-api: enable tests)
Date: Fri, 08 Sep 2017 18:44:02 +0000

Your message dated Fri, 8 Sep 2017 20:42:42 +0200
with message-id <address@hidden>
and subject line Re: [bug#28250] slf4j-api: enable tests
has caused the debbugs.gnu.org bug report #28250,
regarding slf4j-api: enable tests
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden)


-- 
28250: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=28250
GNU Bug Tracking System
Contact address@hidden with problems
--- Begin Message --- Subject: slf4j-api: enable tests Date: Sun, 27 Aug 2017 13:31:42 +0200
Hi,

I found what prevented the tests to pass in slf4j-api. In
slf4j-api/pom.xml, the jar file is created with all classes, except
those in impl/.

I did the same with a new pase where I delete the impl/ directory and
regenerate the jar file. Maybe I should fix teh build.xml file instead?

Then, tests pass except for one file. This file is an abstract class
meant to be extended by other test cases, but it's not a test case in
itself, hence the failure. This time, I implemented another phase where
I fix the build.xml file to exclude this file from testing.

I'm working on other packages, and it's not the first time I see some
tests I need to disable because they shouldn't be run. Maybe I could
add arguments to the ant build system, such as #:test-include and
#:test-exclude, defaulting to '("**/*.java") and '() respectively?

Thank you :)

Attachment: 0001-gnu-java-slf4j-api-Fix-tests.patch
Description: Text Data


--- End Message ---
--- Begin Message --- Subject: Re: [bug#28250] slf4j-api: enable tests Date: Fri, 8 Sep 2017 20:42:42 +0200
Le Fri, 08 Sep 2017 18:05:25 +0200,
address@hidden (Ludovic Courtès) a écrit :

> Julien Lepiller <address@hidden> skribis:
> 
> >>From 2a9b0420c249b4b896f5980a55be8ae5595bd96b Mon Sep 17 00:00:00
> >>2001  
> > From: Julien Lepiller <address@hidden>
> > Date: Sun, 27 Aug 2017 13:19:31 +0200
> > Subject: [PATCH] gnu: java-slf4j-api: Fix tests.
> >
> > * gnu/packages/java.scm (java-slf4j-api)[arguments]: Enable tests
> > Adjust the jar content to prevent a test failure.  
> 
> I don’t fully understand what’s at play but this looks like a step in
> the right direction to me, so I’d say go ahead!
> 
> Thanks,
> Ludo’.

Actually, since there was no answer on this bug, I already pushed it a
few days ago. I forgot to close the ticket.

Pushed as dcf7a8a660554eb952e6412637e26bb523af03e8.


--- End Message ---

reply via email to

[Prev in Thread] Current Thread [Next in Thread]