Skip to content

Instantly share code, notes, and snippets.

@musketyr

musketyr/13af531d-26ec-4ca6-a4b9-55c2f3c3ed7e-sectioned-diff-sc185883.html

Created February 11, 2026 10:05
Show Gist options

  • Select an option

    Learn more about clone URLs
  • Save musketyr/858a0064ab8df912071fec6648571800 to your computer and use it in GitHub Desktop.

Select an option

Learn more about clone URLs
Save musketyr/858a0064ab8df912071fec6648571800 to your computer and use it in GitHub Desktop.
Download ZIP
Side-by-side diff for sc185883 - RoiPeriodicMetricSpec migration
Raw
13af531d-26ec-4ca6-a4b9-55c2f3c3ed7e-sectioned-diff-sc185883.html
<!DOCTYPE html>
<html lang="en">
<head>
<meta charset="UTF-8">
<meta name="viewport" content="width=device-width, initial-scale=1.0">
<title>Spock → JUnit 5 Migration: RoiPeriodicMetricSpec</title>
<style>
* { box-sizing: border-box; margin: 0; padding: 0; }
body { font-family: -apple-system, BlinkMacSystemFont, 'Segoe UI', Roboto, sans-serif; background: #f5f5f5; padding: 20px; line-height: 1.6; }
h1 { text-align: center; margin-bottom: 20px; color: #333; }
.section { background: white; border-radius: 8px; margin-bottom: 20px; box-shadow: 0 2px 4px rgba(0,0,0,0.1); overflow: hidden; }
.section-header { background: #2d3748; color: white; padding: 12px 16px; cursor: pointer; display: flex; justify-content: space-between; align-items: center; }
.section-header:hover { background: #4a5568; }
.section-header h2 { font-size: 16px; font-weight: 600; }
.section-header .toggle { font-size: 12px; }
.section-content { display: block; }
.section-content.collapsed { display: none; }
.diff-container { display: grid; grid-template-columns: 1fr 1fr; }
.diff-panel { padding: 16px; overflow-x: auto; }
.diff-panel.left { background: #fff5f5; border-right: 1px solid #e2e8f0; }
.diff-panel.right { background: #f0fff4; }
.diff-panel h3 { font-size: 12px; text-transform: uppercase; color: #718096; margin-bottom: 12px; letter-spacing: 0.5px; }
pre { font-family: 'SF Mono', Monaco, 'Courier New', monospace; font-size: 13px; white-space: pre-wrap; word-wrap: break-word; }
.left pre { color: #c53030; }
.right pre { color: #276749; }
.summary { background: #c6f6d5; border-left: 4px solid #38a169; padding: 12px 16px; margin: 0; }
.summary p { color: #276749; font-size: 14px; }
.summary strong { color: #22543d; }
@media (max-width: 768px) { .diff-container { grid-template-columns: 1fr; } .diff-panel.left { border-right: none; border-bottom: 1px solid #e2e8f0; } }
</style>
</head>
<body>
<h1>🦀 Spock → JUnit 5 Migration: RoiPeriodicMetricSpec</h1>
<p style="text-align: center; color: #666; margin-bottom: 20px;">Story: <a href="https://app.shortcut.com/agorapulse/story/185883">sc185883</a> | PR: <a href="https://github.com/agorapulse/platform/pull/72702">#72702</a></p>
<!-- Section: Imports -->
<div class="section">
<div class="section-header" onclick="toggleSection(this)">
<h2>📦 Imports</h2>
<span class="toggle">▼</span>
</div>
<div class="section-content">
<div class="diff-container">
<div class="diff-panel left">
<h3>Spock (Groovy)</h3>
<pre>package agorapulse.roi.core.models
import spock.lang.Specification</pre>
</div>
<div class="diff-panel right">
<h3>JUnit 5 (Java)</h3>
<pre>package agorapulse.roi.core.models;
import org.junit.jupiter.api.Test;
import java.math.BigDecimal;
import static org.assertj.core.api.Assertions.assertThat;</pre>
</div>
</div>
<div class="summary">
<p><strong>Changes:</strong> Replaced Spock Specification import with JUnit 5 @Test and AssertJ assertions. Added explicit BigDecimal import.</p>
</div>
</div>
</div>
<!-- Section: Class & Fields -->
<div class="section">
<div class="section-header" onclick="toggleSection(this)">
<h2>🏗️ Class & Fields</h2>
<span class="toggle">▼</span>
</div>
<div class="section-content">
<div class="diff-container">
<div class="diff-panel left">
<h3>Spock (Groovy)</h3>
<pre>class RoiPeriodicMetricSpec extends Specification {</pre>
</div>
<div class="diff-panel right">
<h3>JUnit 5 (Java)</h3>
<pre>class RoiPeriodicMetricTest {</pre>
</div>
</div>
<div class="summary">
<p><strong>Changes:</strong> Removed Specification inheritance (no mocking needed). Renamed Spec → Test.</p>
</div>
</div>
</div>
<!-- Section: Tests -->
<div class="section">
<div class="section-header" onclick="toggleSection(this)">
<h2>🧪 Tests</h2>
<span class="toggle">▼</span>
</div>
<div class="section-content">
<div class="diff-container">
<div class="diff-panel left">
<h3>Spock (Groovy)</h3>
<pre>void 'should get long count'() {
given:
RoiPeriodicMetric metric = RoiPeriodicMetric.builder().count(BigDecimal.TEN).build()
when:
Long count = metric.getLongCountOrZero()
then:
count == 10L
}
void 'should get zero if count null'() {
given:
RoiPeriodicMetric metric = RoiPeriodicMetric.builder().build()
when:
Long count = metric.getLongCountOrZero()
then:
count == 0L
}</pre>
</div>
<div class="diff-panel right">
<h3>JUnit 5 (Java)</h3>
<pre>@Test
void shouldGetLongCount() {
// given
RoiPeriodicMetric metric = RoiPeriodicMetric.builder().count(BigDecimal.TEN).build();
// when
Long count = metric.getLongCountOrZero();
// then
assertThat(count).isEqualTo(10L);
}
@Test
void shouldGetZeroIfCountNull() {
// given
RoiPeriodicMetric metric = RoiPeriodicMetric.builder().count(null).build();
// when
Long count = metric.getLongCountOrZero();
// then
assertThat(count).isZero();
}</pre>
</div>
</div>
<div class="summary">
<p><strong>Changes:</strong> Added @Test annotations. Converted Spock blocks to comments. Replaced == assertions with AssertJ assertThat(). Made count explicitly null in second test for clarity.</p>
</div>
</div>
</div>
<script>
function toggleSection(header) {
const content = header.nextElementSibling;
const toggle = header.querySelector('.toggle');
content.classList.toggle('collapsed');
toggle.textContent = content.classList.contains('collapsed') ? '▶' : '▼';
}
</script>
</body>
</html>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment