Bump log4j2 test dependencies

Change-Id: Idb14af0192ec620dbe334adefbffb64a92bc19ab
Signed-off-by: Kienan Stewart <kstewart@efficios.com>

Add .gitreview

Signed-off-by: Kienan Stewart <kstewart@efficios.com>

Add 'log4j2' domain tests to the Log4j 2.x agent

The Log4j 2.x agent now supports registering with 2 different tracing
domains, the existing 'log4j' domain and the new 'log4j2' domain.

The following tags were added to filter the tests when running the test
suite :

  - agent:jul
  - agent:log4j
  - agent:log4j2
  - domain:jul
  - domain:log4j
  - domain:log4j2
  - benchmark

For example, we can exclude all the log4j2 agent tests when testing a
version of LTTng that doesn't implement it :

  mvn clean verify -Dgroups='!agent:log4j2'

Or to run only the log4j domain tests on all agents that include them :

  mvn clean verify -Dgroups='domain:log4j'

Signed-off-by: Michael Jeanson <mjeanson@efficios.com>

Migrate to Junit 5 Jupiter

This is mostly boilerplate stuff to move from Junit 4 to 5. The only
exceptions are the TestPrintRunner which was converted to an extension
and the parametrized tests that required more involved changes.

Signed-off-by: Michael Jeanson <mjeanson@efficios.com>

Bump test dependencies

Use current versions of the Maven testing framework plugins.

Signed-off-by: Michael Jeanson <mjeanson@efficios.com>

Add log4j2 agent tests

Signed-off-by: Michael Jeanson <mjeanson@efficios.com>

Import project in Eclipse 4.22

Signed-off-by: Michael Jeanson <mjeanson@efficios.com>

Adapt tests to babeltrace 2 output.

The application context type is not present when using babeltrace2 as of
today. This change might get reversed as needed in the future if the
omission of the type was an error on the reader side. For now remove
the type from the lookup and validate based on value only.

Signed-off-by: Jonathan Rajotte <jonathan.rajotte-julien@efficios.com>

Improve observability of checkForLttngTools for debugging

Signed-off-by: Jonathan Rajotte <jonathan.rajotte-julien@efficios.com>

Document how to ignore SIGSEV for debugging

README.md formatting

Add debugging example to ease life of non-java master

Signed-off-by: Jonathan Rajotte <jonathan.rajotte-julien@efficios.com>

Bind the tests-common dependency to test <scope> only

It should only be used by test code.

Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>

Remove Guava dependency

It isn't even used! Easy to re-add if we ever need to, latest version is
21.0.

Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>

Add -SNAPSHOT suffix to version number

Master should annouce a SNAPSHOT version, from which we can then make
"real" releases. Bump to 1.1.0 at the same time.

Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>

Remove some unused imports

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Update Maven plugins versions

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Add new benchmark with no loggers at all

Not extremely representative because of optimizations/JIT, as
the average time diminishes slightly if the number of iterations
increase. It can still give a rough idea of the impact of using
empty Loggers.

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Clear loggers both before AND after tests

Let's try this, it solved the problem in another case...

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Clear loggers at the beginning of tests, not at the end

That way the very first test will also get a clean state.

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Clear the logger configuration before "list" tests

Loggers defined in previous tests often leak into these ones.

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Add workaround in log4j legacy API test

log4j doesn't seem to make it possible clear loggers the way
JUL does, so all loggers defined in previous tests are still
present in subsequent ones.

This coupled with the fact that the legacy agent attaches one
handler to the root logger, all loggers defined in previous
tests end up being visible by LTTng.

Ease up the passing requirement for a test so that it can
still work in these conditions.

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Do not use hierarchical logger names for standard tests

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Add base and JUL tests to test logger hierarchy

Make sure "lttng list" correctly includes parent/child
loggers that may be visible by UST, even if they themselves
do not have LTTng log handlers attached.

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Add tests for "lttng list" when using the legacy agent

Highlights a bug in current master.

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Use a List instead of a Set for listEvents()

This way it will catch if there incorrectly are duplicate events.

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Test with context and retriever names with . and _

Note that LTTng will print periods as underscores in the trace,
which is known and fine, we just have to take it into consideration.

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Add test for invalid context names

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Use >7-bit characters in event names

This will test the string encoding/decoding mechanisms more
thoroughly.

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Update for new LTTngAgent.dispose() method

The method was changed back to being non-static (to not break the
ABI with the old 2.7 class), so update our test code accordingly.

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Put common setup code in the utils classes

We now have a large amount of test classes, it's a bit simpler
to put the common, API-specific @BeforeClass and @AfterClass
code in the utils classes.

At the same time we can give slightly more information in case
of failure of these checks.

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Allow the TcpClientIT test to work with any sessiond

Instead of hard-coding for a root sessiond, we can try root
then fall-back on a user sessiond for these tests.

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Replace test skipping with straight failures

Chasing after skipped tests is annoying. If the environment is
not set correctly, that should still count as a failure.

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Attach exception messages to JUnit fail() calls

Makes failures more informative!

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Document how to set non-default library paths

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Add tests for filter notification ordering

There was a bug in the Java agent with regards to agent teardown:
upon disconnection, the list of tracked events was cleared but no
corresponding filter change notifications were sent! This could leave
some event rules dangling, and future listeners would get incorrectly
notified that these events were enabled while they were not.

Add some tests to cover for this use case.

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Fix latest tests

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Add tests for context-related filtering

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Add tests for application context retrieval

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Add test to to enable events then destroying the session

Upon session destroy, the sessiond should send the corresponding
agent messages to disable the events that were present in the session.

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Add generic context test

We can test context retriever registration/unregistration
independently of the logging APIs.

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Update for new context APIs

No new tests yet, those will come later!

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Add tests for localized messages

This does not apply to the log4j (1.2.x) API, but may apply to
future ones.

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Split the tests package in 3 separate artifacts

Instead of one test package containing both the JUL and log4j tests,
we can define three different ones (-common, -jul, -log4j, similar to
the runtime packages) so that the code only depend on its minimal set
of dependencies.

Will also make it easier to integrate in the runtime project once we
move that to Maven too...

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Handle filter strings being passed by the sessiond

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Fix typo

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Add tests for the "lttng list" command

Should cover https://bugs.lttng.org/issues/933

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Update for new ILttngTcpClientListener method name

listEnabledEvents() was renamed to listAvailableEvents().

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Update for new class names

LogLevelSelector, FilterChangeNotifier

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Rework filter tests to use in-same-thread notifications

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Add a couple filter listener reg/dereg tests

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Move some tests to the "events" subpackage

Better clarify these now that we have "client" and "filter" tests too.

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Add filter notifications and TCP client tests

This required adding log level and filter string functionality
to ILttngSession.

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Use maven-failsafe-plugin instead of surefire to run integration tests

Failsafe is slightly more geared towards integration tests:
it runs in the "verify" phase instead of the "test" one, and
will not fail the build if a test fails, but will report it
at the end.

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Split the lttng-tools wrapper in a separate artifact

This will better isolate dependencies of each component. In
the future it could be moved to a separate git repository, but
for now it's easier to have all the LTTng Java stuff in the
same place.

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Detach the test name printing listener after the test is run

Or else they keep piling up, and the message "Now running test XYZ"
is displayed several times.

Provide default createSession() factory method

Users may not care what backend is actually used, we can provide
a default one.

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Move the "LTTng control" to separate packages

This could move to its own project eventually?

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Isolate the functions for executing shell commands in a new ShellUtils

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Add a TestRunner that will print test names to stdout

Eases debugging a bunch.

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Echo the commands being executed

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Add Javadoc and related settings

Always better to clearly identify what test does what.

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Add copyright header to all source files

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Actually add the profile to run the benchmarks

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Add a README and a LICENSE

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Convert to a Maven project

That way it can run in other environments than the author's
Eclipse workspace.

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Add multi-session tests

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Turn LttngSession(Control) into a non-static class

By managing session objects, the session management becomes much
less verbose in the tests. Additionally, it becomes clear that
the @Before method has to create the session, and the @After has
to close it. And AutoCloseable rocks, as usual.

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Add log4j and legacy-API tests

We can re-use the existing tests into an abstract class.

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Add some integration tests for JUL

as well as more functionality to LttngSessionControl.

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Rename the project to a general ust-java-tests

This can hold both unit/integration tests as well as
benchmarks. Reshuffle the package names a bit to do so.

Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Add benchmarks for old LTTng-JUL API vs. new one

Also move the tests in subpackages.

Switch to use the Lttng-Ust Java Agent API, refactor using JUnit

Commit latest version

Initial commit