polygene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Niclas Hedhman <nic...@hedhman.org>
Subject Re: yeoman-work back to develop
Date Sun, 04 Jun 2017 00:29:36 GMT
Thanks for this.

FYI, when I work on individual issues, I don't rely on test runs, because
they are generally too slow and as you said it isn't 'any' combination.

Instead I use a model json file, and run

    rm -rf * ; yo polygene --import=../model.json --noPrompt

which is mounted in IDE. There I can edit the generation to what I want it
to do, then paste in the resulting code back to the templates and run again.

But a few days ago, I was thinking about that format, and whether I should
change to what the toJson() in tools/model-detail/ generates. I think it is
the right thing to do, but it will take a while, and I am not convinced
that the toJson() will "remain", as I am kind of questioning the whole
visitor pattern and the extreme overlap with the Model itself, it feels
like repetition.

On Sat, Jun 3, 2017 at 8:24 PM, Paul Merlin <paulmerlin@apache.org> wrote:

> Hey,
> I took a quick look at the project generator integration tests.
> First thing, I just tried on a computer where I don't have docker
> installed and the tests present in generated projects simply fail with a
> not so obvious error.
> We should not require docker for a successful build of generated
> projects. This would mean that the tests should be skiped/ignored if a
> docker service is not available.
> We should also not use `latest` tags for docker images otherwise we
> can't guarantee that the generated projects will work in the future.
> About Memcached, I noticed that the docker image used is
> `memcached:latest`. The extension tests (as opposed to tests of
> generated projects) use the Alpine Linux variant instead, see
> https://github.com/apache/polygene-java/blob/develop/
> dependencies.gradle#L281,
> maybe there's something different here.
> Lastly, the docker rule for MongoDBEntityStore is missing. Note that the
> extension tests use an embedded version of MongoDB.
> I've been quite busy this week. I'll have some time early next week to
> try to fix stuff and roll a second release candidate for 3.0 if we are
> ready.
> Given that the tests take quite a long time to run it's unpleasant to
> not be able to easily run a single permutation while working on the
> underlying plumbing. Even the reduced set of tests run by
> :tools:generator:npmTest takes several minutes. Plus I miss proper test
> reports with output and involved paths. Moreover, these tests are
> unfortunately currently disabled on CI. We can look at these problems
> post 3.0.
> Cheers
> /Paul
> Niclas Hedhman a écrit :
> > Gang,
> > I just merged back all the work that has been done on yeoman-work branch.
> >
> > I think the build is Ok, the "checkDist" worked ok locally, think I tried
> > with clean local setup.
> >
> > Well, if it doesn't I will deal with it tomorrow, unless someone beats me
> > to it.
> >
> >
> > The main thing outstanding is that generated testcase is marked @Ignore,
> > until I can figure out all the remaining Docker starts needed. For
> > instance, Memchched is used for most tests, and if I start an memcached
> > server locally, many of the tests works, but with the Docker start that I
> > am attempting, something is not right.
> >
> > Other thing that was fixed was the Configuration bootstrapping, now using
> > the Module of the ConfigurationComposite for any Deserializer service
> > needed.
> >
> > The stacktrace addition is there as well.
> >
> > And probably more that I have forgotten about. For instance, some name
> > changes to try to stay unified and simplified code generation.
> >
> >
> > All-in-all, I think the code generation facility is a good step towards a
> > really great way to get started, without the initial struggles that we
> all
> > face in the beginning to get structure working.
> >
> >
> > Cheers

Niclas Hedhman, Software Developer
http://polygene.apache.org - New Energy for Java

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message