jalview/test/jalview

util

Clone Tools

Stats

Commits this week: 0
Total Committers: 11
Last Commit:
Commits this week: 0

Lines of code count not available

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated a few seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Tuesday 08 Mar
9:58 pm

Jim Procter <j.procter@dundee.ac.uk>

JAL-3746 apply copyright to tests

Options
    • -0
    • +20
    ./JSONUtilsTest.java
    • -0
    • +20
    ./MathUtilsTest.java
    • -0
    • +20
    ./MessageManagerTest.java
    • -0
    • +20
    ./SetUtilsTest.java
    • -0
    • +20
    ./dialogrunner/DialogRunnerTest.java
    • -0
    • +20
    ./matcher/ConditionTest.java
    • -0
    • +20
    ./matcher/MatcherTest.java
  8. … 67 more files in changeset.
9:30 pm

Jim Procter <j.procter@dundee.ac.uk>

JAL-3438 spotless for 2.11.2.0

Options
    • -14
    • +16
    ./ColorUtilsTest.java
    • -12
    • +19
    ./ComparisonTest.java
    • -26
    • +40
    ./DBRefUtilsTest.java
    • -3
    • +4
    ./DnaUtilsTest.java
    • -4
    • +4
    ./FormatTest.java
    • -3
    • +3
    ./MapListTest.java
    • -2
    • +4
    ./MappingUtilsTest.java
    • -3
    • +4
    ./ParseHtmlBodyAndLinksTest.java
    • -1
    • +2
    ./PlatformTest.java
    • -24
    • +24
    ./QuickSortTest.java
    • -0
    • +1
    ./SparseCountTest.java
    • -29
    • +34
    ./StringUtilsTest.java
    • -20
    • +19
    ./UrlLinkTest.java
    • -1
    • +2
    ./dialogrunner/DialogRunnerTest.java
    • -5
    • +5
    ./matcher/MatcherTest.java
  16. … 647 more files in changeset.
Wednesday 02 Mar
6:25 pm

Jim Procter <j.procter@dundee.ac.uk>

JAL-3438 - all src and test run through spotless..

Options
    • -14
    • +16
    ./ColorUtilsTest.java
    • -12
    • +19
    ./ComparisonTest.java
    • -26
    • +40
    ./DBRefUtilsTest.java
    • -3
    • +4
    ./DnaUtilsTest.java
    • -4
    • +4
    ./FormatTest.java
    • -3
    • +3
    ./MapListTest.java
    • -2
    • +4
    ./MappingUtilsTest.java
    • -3
    • +4
    ./ParseHtmlBodyAndLinksTest.java
    • -1
    • +2
    ./PlatformTest.java
    • -24
    • +24
    ./QuickSortTest.java
    • -0
    • +1
    ./SparseCountTest.java
    • -29
    • +34
    ./StringUtilsTest.java
    • -20
    • +19
    ./UrlLinkTest.java
    • -1
    • +2
    ./dialogrunner/DialogRunnerTest.java
    • -5
    • +5
    ./matcher/MatcherTest.java
  16. … 654 more files in changeset.
Friday 11 Feb
4:12 pm

Ben Soares

That looks good. I was almost going to do something similar (I was thinking jalview.log.Log, but jalview.bin.Console is just as good) so happy with this!

That looks good. I was almost going to do something similar (I was thinking jalview.log.Log, but jalview.bin.Console is just as good) so happy with this!

3:49 pm

Ben Soares

Added those in. I'm still converting to String with getCanonicalName in the getLogger(Class ...) method because, e.g. JLogger jlogger1 = JLogger.getLogger(Jalview.class); JLogger jlogger2 = JLogger...

Added those in. I'm still converting to String with getCanonicalName in the getLogger(Class ...) method because, e.g.
JLogger jlogger1 = JLogger.getLogger(Jalview.class);
JLogger jlogger2 = JLogger.getLogger("jalview.bin.Jalview");
should really return an existing logger in the registry if it exists, and the String name is the registry key.

3:40 pm

Ben Soares

I was thinking that JLogger did this comparison anyway, but I've followed it through and it didn't. So I've added if (loglevel.compareTo(this.level) < 0) { return false; } to JLogger.println so t...

I was thinking that JLogger did this comparison anyway, but I've followed it through and it didn't. So I've added

if (loglevel.compareTo(this.level) < 0) { return false; }

to JLogger.println so that a comparison is made between the (enums) of JLogger.level and the message LogLevel, and if it's not going to be logged then it doesn't call the actual logger at all (in this case Log4j, so the log4j object is never even touched [even with logger.isDebugEnabled() because the answer should be no]).
This now applies to all log calls.
The check whether Cache.log != null is equivalent to Cache.log.loggerExists() which already happens within JLoggerLog4j.
There is admittedly the wrapper of the JLogger getting brought to forefront of memory, but that's presumably smaller than the old Cache.log which was a Log4j.Logger object.

3:27 pm

Jim Procter <j.procter@dundee.ac.uk>

JAL-3949 - refactor logging from jalview.bin.Cache to jalview.bin.Console

Conflicts:

src/jalview/bin/Jalview.java

Options
    • -3
    • +3
    ./Log4jTest.java
    • -2
    • +2
    ./MapListTest.java
    • -2
    • +2
    ./MappingUtilsTest.java
  4. … 96 more files in changeset.
2:15 pm

Jim Procter

2:09 pm

Jim Procter

AWESOME JOB !!!! one tiny niggle I had was that Cache.debug/Cache.error doesn't read nicely, so I've abstracted the logging stuff even farther to a new jalview.bin.Console class: see https://source...

AWESOME JOB !!!! one tiny niggle I had was that Cache.debug/Cache.error doesn't read nicely, so I've abstracted the logging stuff even farther to a new jalview.bin.Console class: see https://source.jalview.org/crucible/changelog/~br=update%402fJAL-3949_jalview_bin_Console_to_cure_sanity_even_more/jalview/

1:49 pm

Jim Procter

the reason for conditionals is that Java won't perform the string append operation if logLevel>debug. It saves a minor amount of processing time which is important for code that is executed lots.

the reason for conditionals is that Java won't perform the string append operation if logLevel>debug. It saves a minor amount of processing time which is important for code that is executed lots.

1:46 pm

Jim Procter

You could have .getLogger(Class ..) rather than having to call getCanonicalName ?

You could have .getLogger(Class ..) rather than having to call getCanonicalName ?

1:36 pm

Jim Procter

JAL-3949 An attempt at converting to Log4j 2 -- no output achieved!
JAL-3949 An attempt at converting to Log4j 2 -- no output achieved!
3:34 am

Ben Soares

JAL-3949 Complete new abstracted logging framework in jalview.log. Updated log calls for this. Removal of jalview.javascript.log4j. Improvements to the Java Console (controllable autoscrolling).

Options
    • -0
    • +199
    ./Log4jTest.java
    • -0
    • +15
    ./log4jTestProps.jvprops
  3. … 118 more files in changeset.
3:32 am

Jim Procter

Tuesday 08 Feb
11:42 am

Jim Procter

I'm abandoning this review for now (see my comments on the issue). For what its worth since this is a string literal you should at least follow convention JALVIEW_LOGGER rather than a generic 'LOGG...

I'm abandoning this review for now (see my comments on the issue). For what its worth since this is a string literal you should at least follow convention JALVIEW_LOGGER rather than a generic 'LOGGER_NAME'. The code for routing logged content from other loggers should probably be a method on Cache... or even better, separate all this out into the jalview.util.Log4j class so Cache doesn't get polluted with log configuration cruft.

11:24 am

Ben Soares

At the moment it is only used in jalview.bin.Cache, but it is used in separate places where appenders are added. Previous to this it was relying on the String name of the appender being correctly u...

At the moment it is only used in jalview.bin.Cache, but it is used in separate places where appenders are added. Previous to this it was relying on the String name of the appender being correctly used. However the Strings that were being used were not the same so it was broken (I don't know how that manifested itself, but it definitely would not have worked the way it used to!).
Feasibly a separate logger made in the future (e.g. API logger?) will want to add this Appender and will likely not be made in jalview.bin.Cache, so putting a constant String somewhere generically connected with Log4j seemed sensible. Correct it needs documenting!

10:10 am

Jim Procter

Does this really need to be a string constant ? (I'm not convinced) needs documenting as the log4j internal name for the log if it really needs to be external.

Does this really need to be a string constant ? (I'm not convinced) needs documenting as the log4j internal name for the log if it really needs to be external.

10:05 am

Jim Procter

JAL-3949 An attempt at converting to Log4j 2 -- no output achieved!
JAL-3949 An attempt at converting to Log4j 2 -- no output achieved!
2:14 am

Ben Soares

Test to check for debug Log4j Test message

Options
    • -0
    • +199
    ./Log4jTest.java
    • -0
    • +15
    ./log4jTestProps.jvprops
Friday 04 Feb
2:34 pm

Jim Procter <j.procter@dundee.ac.uk>

Merge branch 'bug/JAL-3806_mappingCoversSequence_for2112' into develop Combines patches for JAL-3806 prior to redaction of shared CDS dataset sequences with patches for JAL-3673 Conflicts: src/jalview/datamodel/AlignedCodonFrame.java src/jalview/util/MappingUtils.java test/jalview/util/MappingUtilsTest.java

Options
    • -10
    • +0
    ./MappingUtilsTest.java
  2. … 2 more files in changeset.
Friday 28 Jan
5:08 pm

Jim Procter <j.procter@dundee.ac.uk>

Merge branch 'merge/develop_JAL-3725' into develop

Conflicts:

src/jalview/gui/PopupMenu.java

Options
    • -52
    • +138
    ./MapListTest.java
    • -25
    • +15
    ./MappingUtilsTest.java
  3. … 2 more files in changeset.
Wednesday 26 Jan
7:36 pm

Jim Procter <j.procter@dundee.ac.uk>

Merge branch 'task/JAL-3763_newDatasetForCds' into merge/develop_task/JAL-3763_newDatasetForCds

Conflicts:

src/jalview/analysis/AlignmentUtils.java

src/jalview/util/MapList.java

test/jalview/util/MapListTest.java

test/jalview/util/MappingUtilsTest.java

Options
    • -3
    • +4
    ./MapListTest.java
    • -2
    • +114
    ./MappingUtilsTest.java
  3. … 5 more files in changeset.
Tuesday 23 Nov 2021
3:54 pm

Mateusz Warowny

JAL-3066 Fix ambiguous assertEquals casting values to double.

Options
    • -17
    • +17
    ./HMMProbabilityDistributionAnalyserTest.java
Thursday 16 Sep 2021
10:24 am

Ben Soares

JAL-3761 listToArray became rangeListToArray

Options
    • -169
    • +204
    ./MappingUtilsTest.java
Sunday 25 Jul 2021
4:23 pm

Jim Procter <j.procter@dundee.ac.uk>

JAL-2994 disable tests for FileUtils path search methods on bamboo

Options
    • -2
    • +8
    ./FileUtilsTest.java
Tuesday 06 Jul 2021
5:42 pm

Jim Procter

5:42 pm

Jim Procter

WIP
WIP
Thursday 04 Mar 2021
11:44 am

Jim Procter

Revert "Merge branch 'bug/JAL-3806_mappingCoversSequence' into releases/Release_2_11_1_Branch"

This reverts commit cb628aab46f9027e71b71203b322f96710e09990, reversing

changes made to a40669b2eeb34a6c760b63c172a31a194b456f24.

Options
    • -111
    • +20
    ./MappingUtilsTest.java
  2. … 2 more files in changeset.
Thursday 04 Feb 2021
3:09 pm

gmungoc <g.m.carstairs@dundee.ac.uk>

JAL-3806 fix and test for one-to-many sequence mapping case

Options
    • -20
    • +111
    ./MappingUtilsTest.java
  2. … 1 more file in changeset.
Wednesday 02 Dec 2020
8:37 pm

gmungoc <g.m.carstairs@dundee.ac.uk>

JAL-3751 second occurrence of bug squashed

Options
    • -22
    • +34
    ./MapListTest.java
  2. … 1 more file in changeset.