Skip to content

A7: Convert TryFinallyTest to JUnit4 with assertions #495

Closed
#505
Closed
A7: Convert TryFinallyTest to JUnit4 with assertions#495
#505
Labels
agent/doneThe agent finished successfully; PR opened or comment posted.area/corecore/ — VM, kernel, scheduler, classmgr, drivers.area/testUnit tests, boot tests, QEMU test infrastructure.kind/featureNew feature or enhancement request.
@LSantha

Description

@LSantha
LSantha
opened on Jun 7, 2026

Goal

Convert TryFinallyTest to JUnit4 with proper assertions for finally semantics.

Source Under Test

  • File: JVM try-finally semantics
  • Class: N/A (language feature test)
  • Purpose: Verify finally block execution

Current State

  • Test file: core/src/test/org/jnode/test/TryFinallyTest.java
  • Type: JUnit3/Legacy
  • Quality: no-assertions (1 method, 0 assertions)
  • Requires JNode: No (host JDK runnable)

Approach

  1. Convert to JUnit4
  2. Add assertions for finally execution on: normal return, exception throw, break/continue/return in try
  3. Add to CoreTestSuite
  4. Verify test compiles and passes

Acceptance Criteria

  • Test class compiles
  • sh build.sh tests exits 0
  • JUnit XML report contains the test
  • QEMU boot test passes
  • PR opened against master

Metadata

Metadata

Assignees

No one assigned

    Labels

    agent/doneThe agent finished successfully; PR opened or comment posted.area/corecore/ — VM, kernel, scheduler, classmgr, drivers.area/testUnit tests, boot tests, QEMU test infrastructure.kind/featureNew feature or enhancement request.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions