Hi,

using them in the intergration test originally was just a "hey, let's cover
those things also in our integration tests", so originally it was the other
way around: standalone dev-tools that we covered in our tests a bit.
Having them as standalone tools in "examples" sounds nice, as they still
can full-fill a small and defined purpose outside of tests, maybe also for
some of our more experienced users.

Dominik.


On Thu, Nov 25, 2021 at 10:01 PM Andreas Beeker <[email protected]>
wrote:

> Hi Dominik,
>
> there's a replacement, but with a limitation.
>
> the limitation: asking users to execute BiffViewer with the poi.jar won't
> work anymore.
> I can't remember having seen this request ever.
>
> In other usecases - use the Junit c<lasses in poi::test::
> org.apache.poi.hssf.dev directly.
> Those contain now the functionality originally provided by the poi::main
> classes.
> Overriding BaseTestIteratingXLS::files() makes it easy to define the files
> to be parsed.
> I usually prefer test drivers to main() invocations and also use test
> drivers when developing examples.
>
> For the visual inclined (aka Klickibunti), there's another alternative:
> https://github.com/kiwiwings/poi-visualizer
>
> Andi
>
> On 25.11.21 07:32, Dominik Stadler wrote:
> > Hi Andi,
> >
> > It seems you removed the main() method from BiffViewer as part of this
> > move. This was a handy tool on it's own for inspecting the raw contents
> of
> > .xls files, fairly useful when investigating bugs with handling of the
> raw
> > format in HSSF. Is there a replacement?
> >
> > Not sure about the other tools that are gone now, they might be useful
> for
> > some specific things, but at least I did not use them much.
> >
> > Thanks... Dominik.
> >
> > On Sun, Nov 14, 2021 at 3:31 PM Andreas Beeker <[email protected]>
> wrote:
> >
> >> Hi Devs,
> >>
> >> those *-saved.xls left-overs in the test-data directory cause/d build
> >> problems.
> >> I'm quite sure those are created by org.apache.poi.hssf.dev.ReSave.
> >> I'll now move all development utils of that package to poi:test and
> change
> >> the output directory to the temp path, because I think those tools are
> used
> >> by developers working on the source / project ... or I'll remove a few
> >> completely, as the same functionality is invoked many times by the other
> >> tests.
> >>
> >> Andi
> >>
> >>
> >> ---------------------------------------------------------------------
> >> To unsubscribe, e-mail: [email protected]
> >> For additional commands, e-mail: [email protected]
> >>
> >>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [email protected]
> For additional commands, e-mail: [email protected]
>
>

Reply via email to