25600722838

Committed 09 May 2026 12:06PM UTC coverage: 83.396% (-5.3%) from 88.677%

Build # 25600722838

Build Type

push

github

Committed by

web-flow

Commit Message

Nats v2 web (#295)

* 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 shipped the t... (continued)

Coverage Stats

2566 of 3407 branches covered (75.32%)

Branch coverage included in aggregate %.

795 of 1072 new or added lines in 22 files covered. (74.16%)

312 existing lines in 38 files now uncovered.

7715 of 8921 relevant lines covered (86.48%)

0.86 hits per line

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

70.37

/rqueue-core/src/main/java/com/github/sonus21/rqueue/converter/JsonMessageConverter.java

/*
 * Copyright (c) 2021-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.converter;

import static org.springframework.util.Assert.notNull;

import com.github.sonus21.rqueue.serdes.RqueueSerDes;
import com.github.sonus21.rqueue.serdes.SerializationUtils;
import lombok.extern.slf4j.Slf4j;
import org.springframework.messaging.Message;
import org.springframework.messaging.MessageHeaders;
import org.springframework.messaging.converter.MessageConverter;
import org.springframework.messaging.support.GenericMessage;

/**
 * JsonMessageConverter tries to convert to JSON and from JSON to object.
 *
 * <p>Message converter relies on target class information to deserialize JSON to object. If it
 * finds target class is null then it returns the null.
 *
 * <p>Target class is null till the time method arguments are not resolved, once method arguments
 * are resolved then it will become non-null.
 *
 * @see org.springframework.messaging.converter.JacksonJsonMessageConverter
 */
@Slf4j
public class JsonMessageConverter implements MessageConverter {

  private final RqueueSerDes serDes;

  public JsonMessageConverter() {
    this.serDes = SerializationUtils.getSerDes();
  }

  public JsonMessageConverter(RqueueSerDes serDes) {
    notNull(serDes, "serDes cannot be null");
    this.serDes = serDes;
  }

  @Override
  public Object fromMessage(Message<?> message, Class<?> targetClass) {
    log.trace("Message: {} TargetClass: {}", message.getPayload(), targetClass);
    try {
      String payload = (String) message.getPayload();
      if (targetClass == null) {
        return null;
      }
      if (SerializationUtils.isJson(payload)) {
        return serDes.deserialize(payload, targetClass);
      }
      return null;
    } catch (Exception e) {
      log.debug("Deserialization of message {} failed", message, e);
      return null;
    }
  }

  @Override
  public Message<?> toMessage(Object payload, MessageHeaders headers) {
    log.trace("Payload: {} Headers: {}", payload, headers);
    try {
      return new GenericMessage<>(serDes.serializeAsString(payload));
    } catch (Exception e) {
      log.debug("Serialisation failed, Payload: {}", payload, e);
      return null;
    }
  }
}

1	/*
2	* Copyright (c) 2021-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.converter;
18
19	import static org.springframework.util.Assert.notNull;
20
21	import com.github.sonus21.rqueue.serdes.RqueueSerDes;
22	import com.github.sonus21.rqueue.serdes.SerializationUtils;
23	import lombok.extern.slf4j.Slf4j;
24	import org.springframework.messaging.Message;
25	import org.springframework.messaging.MessageHeaders;
26	import org.springframework.messaging.converter.MessageConverter;
27	import org.springframework.messaging.support.GenericMessage;
28
29	/**
30	* JsonMessageConverter tries to convert to JSON and from JSON to object.
31	*
32	* <p>Message converter relies on target class information to deserialize JSON to object. If it
33	* finds target class is null then it returns the null.
34	*
35	* <p>Target class is null till the time method arguments are not resolved, once method arguments
36	* are resolved then it will become non-null.
37	*
38	* @see org.springframework.messaging.converter.JacksonJsonMessageConverter
39	*/
40	@Slf4j	1✔
41	public class JsonMessageConverter implements MessageConverter {
42
43	private final RqueueSerDes serDes;
44
45	public JsonMessageConverter() {	1✔
46	this.serDes = SerializationUtils.getSerDes();	1✔
47	}	1✔
48
49	public JsonMessageConverter(RqueueSerDes serDes) {	1✔
50	notNull(serDes, "serDes cannot be null");	1✔
51	this.serDes = serDes;	1✔
52	}	1✔
53
54	@Override
55	public Object fromMessage(Message<?> message, Class<?> targetClass) {
56	log.trace("Message: {} TargetClass: {}", message.getPayload(), targetClass);	1✔
57	try {
58	String payload = (String) message.getPayload();	1✔
59	if (targetClass == null) {	1✔
60	return null;	1✔
61	}
62	if (SerializationUtils.isJson(payload)) {	1!
63	return serDes.deserialize(payload, targetClass);	1✔
64	}
UNCOV 65	return null;	×
UNCOV 66	} catch (Exception e) {	×
67	log.debug("Deserialization of message {} failed", message, e);	×
68	return null;	×
69	}
70	}
71
72	@Override
73	public Message<?> toMessage(Object payload, MessageHeaders headers) {
74	log.trace("Payload: {} Headers: {}", payload, headers);	1✔
75	try {
76	return new GenericMessage<>(serDes.serializeAsString(payload));	1✔
UNCOV 77	} catch (Exception e) {	×
UNCOV 78	log.debug("Serialisation failed, Payload: {}", payload, e);	×
UNCOV 79	return null;	×
80	}
81	}
82	}

sonus21 / rqueue / 25600722838

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