On Tue, 16 Nov 2021 05:36:37 GMT, Jonathan Gibbons <j...@openjdk.org> wrote:
> Please review a moderately simple addition to the `JavadocTester` world, to > be able to modify the behavior of a file manager to throw exceptions as > needed. > > The bulk of the new functionality is in a new class, > `TestJavaFileManagerBuilder`, which uses the builder pattern to allow a > client to configure and build a file manager that can provide file objects > for which the behavior can be overridden for one or more methods. The > expected use case is to throw an exception instead of calling the underlying > delegate method. > > `JavadocTester` is modified to allow a file manager to be provided when > invoking `javadoc`. This requires some minor changes to the outermost javadoc > tool classes, `Main` and `Start`. Rather than add more `static` methods to > `Main`, instance methods are now provided to make it easier to configure the > values that will be passed to `Start`. In `Start`, it was previously > assumed that either the default file manager was being used or that _all_ > paths would be configured directly in the file manager. The latter part of > that assumption is reduced so that path-setting options (e.g. > `--source-path`, `--class-path` etc) can be passed to the provided file > manager. (Previously, they were silently ignored.) It is an error to pass > path-like options as javadoc options to a file manager that does not support > them. However, since none of the changes are visible to any public API, this > should not be an issue. > > A new test is provided, with a few simple test cases. One is for direct use > of the new file manager mechanism (without using `javadoc`). The other two > illustrate how the feature can be used inside a `JavadocTester` call of > `javadoc`. Given that the feature is a relatively simple combination of > predicates and proxies, it's not clear that we need a significant body of > test cases. After having looked at this change, I think I might have fallen prey to the famous "XY problem". Instead of asking on how to emulate error conditions in a black-box fashion first, I asked to provide white-box support for those through JavadocTester. Just to make sure we're on the same page and aren't trying to solve a more complex problem than is required. Is there an easy way to make these methods throw: `javax.tools.JavaFileManager#getFileForInput` and `javax.tools.FileObject#getCharContent`? ------------- PR: https://git.openjdk.java.net/jdk/pull/6404
