ponder-lab / Hybridize-Functions-Refactoring / #1653
72%

DEFAULT BRANCH: main
Ran
Jobs 1
Files 31
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 72.219%. Remained the same
#1653

push

travis-pro

web-flow 
Refactor `testHasLikelyTensorParameterHelper` to encapsulate audit assertions (#547)

* Refactor `testHasLikelyTensorParameterHelper` to encapsulate audit assertions

Helper-pattern restoration: the helper name says it does the testing, so it should. The previous design had the structural overload return `Function` so audit overloads could chain off it internally—but that return type leaked through to test fixtures that did inline assertions on the returned `Function`. Eliminate the `Function` return entirely; helpers now take all expectations as parameters and assert everything in one call.

**General helper** (new): `testHasLikelyTensorParameterHelper(boolean expectingHybridFunction, boolean expectingTensorParameter, Set<TensorType> aTensorTypes, Set<TensorType> bTensorTypes, Optional<List<TensorType>> expectedSignature)`. Loads the function, runs the structural check, and (when the per-parameter expectation arguments are non-null) asserts the per-parameter `getTensorTypes()` sets and the `inferInputSignature()` value. Returns void. No `Function` escapes.

**Convenience overloads** all delegate to the general helper, all return void:

- `(boolean, boolean)` — structural-only.
- `(TensorType expected)` — symmetric IDEAL.
- `(boolean, TensorType)` — symmetric IDEAL with explicit hybrid status.
- `(TensorType layer1, TensorType layer2)` — symmetric Layer-1 / Layer-2 distinct (ragged-tensor anchor case).
- `HelperNoTensor(boolean expectingHybridFunction)` — no-tensor case (`getTensorTypes()` empty, signature drops at classification step).
- `HelperMultiContext(Set<TensorType> contexts, TensorType inferred)` — multi-context narrowing.

**Fixture migrations** (named in the design discussion):

- **Fixture 11** (asymmetric per-parameter): switched from inline structural + audit to general helper with explicit `Set.of(expectedA)` / `Set.of(expectedB)` and an `Optional` signature.
- **Fixture 25** (`tf.Tensor` low-level constructor, FLOAT32 scalar): switched... (continued)

1188 of 1645 relevant lines covered (72.22%)

0.72 hits per line

Jobs
ID Job ID Ran Files Coverage
1 #1653.1 31
72.22
Source Files on build #1653