Skip to content

Commit

Permalink
[#1396][#1401] update RELEASE-NOTES, fix failing tests
Browse files Browse the repository at this point in the history
  • Loading branch information
remkop committed Feb 10, 2022
1 parent 98f2647 commit dc4dc7f
Show file tree
Hide file tree
Showing 3 changed files with 36 additions and 6 deletions.
30 changes: 30 additions & 0 deletions RELEASE-NOTES.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,35 @@
# picocli Release Notes

# <a name="4.7.0"></a> Picocli 4.7.0 (UNRELEASED)
The picocli community is pleased to announce picocli 4.7.0.

This release includes bugfixes and enhancements.


This is the seventy-nineth public release.
Picocli follows [semantic versioning](http://semver.org/).
Artifacts in this release are signed by Remko Popma (6601 E5C0 8DCC BB96).

## <a name="4.7.0-toc"></a> Table of Contents
* [New and noteworthy](#4.7.0-new)
* [Fixed issues](#4.7.0-fixes)
* [Deprecations](#4.7.0-deprecated)
* [Potential breaking changes](#4.7.0-breaking-changes)

## <a name="4.7.0-new"></a> New and Noteworthy


## <a name="4.7.0-fixes"></a> Fixed issues
[#1396][#1401] API: Support generic types in containers (e.g. List, Map). Thanks to [Michał Górniewski](https://github.com/mgorniew) for the pull request.



## <a name="4.7.0-deprecated"></a> Deprecations
No features were deprecated in this release.

## <a name="4.7.0-breaking-changes"></a> Potential breaking changes
This release has no breaking changes.


# <a name="4.6.3"></a> Picocli 4.6.3
The picocli community is pleased to announce picocli 4.6.3.
Expand Down
10 changes: 4 additions & 6 deletions src/test/java/picocli/GenericTest.java
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,7 @@
import java.util.List;
import java.util.Map;

import static org.junit.Assert.assertEquals;
import static org.junit.Assert.fail;
import static org.junit.Assert.*;
import static picocli.CommandLine.ITypeConverter;
import static picocli.CommandLine.Parameters;

Expand All @@ -35,7 +34,6 @@ public String toString() {
}

static final class GenericValueConverter implements ITypeConverter<GenericValue<?>> {

public GenericValue<?> convert(String value) throws Exception {
if (value.startsWith("s")) {
return new GenericValue<String>(value);
Expand Down Expand Up @@ -68,7 +66,9 @@ class NoConverterApp {
CommandLine.populateCommand(new NoConverterApp(), "sOne", "15");
fail("Expected exception");
} catch (MissingTypeConverterException ex) {
assertEquals("No TypeConverter registered for picocli.GenericTest$GenericValue of field java.util.List<picocli.GenericTest$GenericValue<?>> picocli.GenericTest$1NoConverterApp.values", ex.getMessage());
String j8errMsg = "No TypeConverter registered for picocli.GenericTest$GenericValue of field java.util.List<picocli.GenericTest.picocli.GenericTest$GenericValue<?>> picocli.GenericTest$1NoConverterApp.values";
String j11errMsg = "No TypeConverter registered for picocli.GenericTest$GenericValue of field java.util.List<picocli.GenericTest$GenericValue<?>> picocli.GenericTest$1NoConverterApp.values";
assertTrue(ex.getMessage(),j8errMsg.equals(ex.getMessage()) || j11errMsg.equals(ex.getMessage()));
}
}

Expand All @@ -80,8 +80,6 @@ class NoConverterApp {
}
@SuppressWarnings("rawtypes")
class GenericConverter implements ITypeConverter<GenericValue> {

@Override
public GenericValue convert(String value) throws Exception {
return new GenericValue("abc");
}
Expand Down
2 changes: 2 additions & 0 deletions src/test/java/picocli/ModelTypedMemberTest.java
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
package picocli;

import org.junit.Ignore;
import org.junit.Rule;
import org.junit.Test;
import org.junit.contrib.java.lang.system.ProvideSystemProperty;
Expand All @@ -21,6 +22,7 @@ public class ModelTypedMemberTest {
@Rule
public final ProvideSystemProperty ansiOFF = new ProvideSystemProperty("picocli.ansi", "false");

@Ignore("No longer expected to fail after [#1396][#1401]")
@Test
public void testInferTypes() {
class App {
Expand Down

0 comments on commit dc4dc7f

Please sign in to comment.