25621809822

Committed 10 May 2026 06:27AM UTC coverage: 83.425% (+0.03%) from 83.396%

Build # 25621809822

Build Type

push

github

Committed by

web-flow

Commit Message

Nats scheduling fix (#297)

* ci: compile main sources in coverage_report job

The coverage_report job was producing an effectively empty
jacocoTestReport.xml (3.4KB vs ~1.1MB locally) because no .class files
existed when coverageReportOnly ran — the job checked out source code
and downloaded .exec artifacts, but never compiled. JaCoCo's report
generator skips packages/classes it cannot resolve, so the merged XML
ended up with only <sessioninfo> entries and no <package> elements.

That made coverallsJacoco silently no-op via the
"source file set empty, skipping" branch in CoverallsReporter, so
"Push coverage to Coveralls" reported success without uploading.

Verified by downloading the coverage-report artifact from a recent run
and comparing its XML structure against a local build's report.

Assisted-By: Claude Code

* nats-web: implement pause / soft-delete admin ops and capability-aware Q-detail

Replace the all-stub `NatsRqueueUtilityService` with real impls for the operations
JetStream can model: `pauseUnpauseQueue` persists the `paused` flag on `QueueConfig`
in the queue-config KV bucket and notifies the local listener container so the poller
stops dispatching; `deleteMessage` is a soft delete via `MessageMetadataService`
(stream message persists, dashboard hides via the metadata flag); `getDataType`
reports `STREAM`. `moveMessage`, `enqueueMessage`, and `makeEmpty` deliberately
remain "not supported" — there is no JetStream primitive for those.

Update `RqueueQDetailServiceImpl.getRunningTasks` / `getScheduledTasks` to return
header-only tables when the broker capabilities suppress those sections, instead of
emitting zero rows or 501s on NATS.

20 new unit tests cover the pause/delete paths and lock in the still-unsupported
operations. Updates `nats-task.md` / `nats-task-v2.md` to reflect what landed.

Assisted-By: Claude Code

* nats-web: capability-aware nav / charts and stream-based peek

End-to-end browser-tested the NATS dashboard and shipp... (continued)

Coverage Stats

2628 of 3487 branches covered (75.37%)

Branch coverage included in aggregate %.

128 of 179 new or added lines in 12 files covered. (71.51%)

1 existing line in 1 file now uncovered.

7841 of 9062 relevant lines covered (86.53%)

0.87 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

0.0

/rqueue-core/src/main/java/com/github/sonus21/rqueue/metrics/RqueueMetricsCounter.java

/*
 * Copyright (c) 2019-2026 Sonu Kumar
 *
 * Licensed under the Apache License, Version 2.0 (the "License");
 * You may not use this file except in compliance with the License.
 * You may obtain a copy of the License at
 *
 *     https://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and limitations under the License.
 *
 */

package com.github.sonus21.rqueue.metrics;

public interface RqueueMetricsCounter {

  void updateFailureCount(String queueName);

  void updateExecutionCount(String queueName);

  /**
   * Consumer-aware failure increment. When a queue carries multiple {@code @RqueueListener}
   * methods with distinct {@code consumerName} overrides, each consumer has its own counter
   * registered under {@code (queueName, consumerName)}; calling the bare-queue overload would
   * route every increment to the same (last-registered) counter and silently lose per-consumer
   * counts. Defaults to the queue-level path so callers that don't have a consumer name keep
   * working unchanged.
   */
  default void updateFailureCount(String queueName, String consumerName) {
    updateFailureCount(queueName);
  }

  /** Consumer-aware execution increment. See {@link #updateFailureCount(String, String)}. */
  default void updateExecutionCount(String queueName, String consumerName) {
    updateExecutionCount(queueName);
  }
}

1	/*
2	* Copyright (c) 2019-2026 Sonu Kumar
3	*
4	* Licensed under the Apache License, Version 2.0 (the "License");
5	* You may not use this file except in compliance with the License.
6	* You may obtain a copy of the License at
7	*
8	* https://www.apache.org/licenses/LICENSE-2.0
9	*
10	* Unless required by applicable law or agreed to in writing, software
11	* distributed under the License is distributed on an "AS IS" BASIS,
12	* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
13	* See the License for the specific language governing permissions and limitations under the License.
14	*
15	*/
16
17	package com.github.sonus21.rqueue.metrics;
18
19	public interface RqueueMetricsCounter {
20
21	void updateFailureCount(String queueName);
22
23	void updateExecutionCount(String queueName);
24
25	/**
26	* Consumer-aware failure increment. When a queue carries multiple {@code @RqueueListener}
27	* methods with distinct {@code consumerName} overrides, each consumer has its own counter
28	* registered under {@code (queueName, consumerName)}; calling the bare-queue overload would
29	* route every increment to the same (last-registered) counter and silently lose per-consumer
30	* counts. Defaults to the queue-level path so callers that don't have a consumer name keep
31	* working unchanged.
32	*/
33	default void updateFailureCount(String queueName, String consumerName) {
NEW 34	updateFailureCount(queueName);	×
NEW 35	}	×
36
37	/** Consumer-aware execution increment. See {@link #updateFailureCount(String, String)}. */
38	default void updateExecutionCount(String queueName, String consumerName) {
NEW 39	updateExecutionCount(queueName);	×
NEW 40	}	×
41	}

sonus21 / rqueue / 25621809822

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous