add missing env informations to block #401

[nadar]

closes #401

[hbugdoll]

I've made new unit tests for the envOptions: https://github.com/luyadev/luya-module-cms/commit/ef4efcd366a6572d4bd1098cb8b4f92fb9dd1ae5.

And they have proven their worth – my 'equalIndex' iteration was wrong (post-increment vs. pre-increment), see https://github.com/luyadev/luya-module-cms/commit/2d3be0da2574b21f4fa29ab030b9e1470e9bca5d.

Block Test

• Added new BlockTest::testBlockEnvOptions()
• Enhanced therefore TestBlock::admin()
  • Accessing and returning envOptions and unused config value
• Improved BlockTest::testBlockValues() (checking unused config value)

Block Placeholder Iteration Test

• Added new test BlockPlaceholderIterationTest::testEnvOptionsPlaceholderIteration()
  • 3 equal blocks on 1 page via fixtures
  • Checking envOptions in admin context via NavItemPage::getPlaceholder()
  • Checking envOptions in frontend context via NavItemPage::renderPlaceholder()
• Added therefore new testing block TestingEnvOptionsBlock
  • Accessing on envOptions in admin context inside renderAdmin()
  • Accessing on envOptions in frontend context inside renderFrontend() (JSON decoding/encoding is needed because of internal processing & concat)
• Completed access on all block models in BlockPlaceholderIterationTest::testRenderPlaceholderIteration() for sake of completeness

[nadar]

@hbugdoll i will merge this PR.. You can send then a PR with only the unit tests and the according fix. ok? Afterwards we will take a look at the page object in another PR.

[nadar]

@hbugdoll you can now create a fresh merge request using the latest master (don't forget to create a branch on your fork).

[hbugdoll]

@hbugdoll you can now create a fresh merge request using the latest master (don't forget to create a branch on your fork).

Ok, done → #407