feat: simplification Renderer->columnOptions output for datetime now

[aquaminer]

Before

            ->addColumn('created_at', 'datetime', [
                'nullable' => false,
                'default'  => \Cycle\Database\Injection\Fragment::__set_state(array(
               'fragment' => 'now()',
               'parameters' =>
              array (
                ),
                ))
            ])

After

            ->addColumn('created_at', 'datetime', [
                'nullable' => false,
                'default'  => 'CURRENT_TIMESTAMP'
               )
            ])

CURRENT_TIMESTAMP will be translated for postgresql to now() in AbstractColumn (cycle/database)

[codecov]

Codecov Report

Merging #33 (f631067) into 3.x (91f6f3e) will decrease coverage by 0.32%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##                3.x      #33      +/-   ##
============================================
- Coverage     95.91%   95.59%   -0.33%     
- Complexity      246      248       +2     
============================================
  Files            29       29              
  Lines           784      726      -58     
============================================
- Hits            752      694      -58     
  Misses           32       32              

Impacted Files	Coverage Δ
src/Atomizer/Renderer.php	100.00% <100.00%> (ø)
src/Migrator.php	83.73% <0.00%> (-0.27%)	⬇️
src/Capsule.php	100.00% <0.00%> (ø)
src/FileRepository.php	100.00% <0.00%> (ø)
src/Operation/Column/Rename.php	100.00% <0.00%> (ø)
src/Operation/ForeignKey/Add.php	100.00% <0.00%> (ø)
src/Operation/ForeignKey/Drop.php	100.00% <0.00%> (ø)
src/Operation/ForeignKey/Alter.php	100.00% <0.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 91f6f3e...f631067. Read the comment docs.

[msmakouz]

I recommend adding a strict comparison here (===).

It might be worth adding a test with new functionality. What do you think? @roxblnfk

[aquaminer]

if use strict comparison here, before/after example don't be work (because Fragment does not converted to string).

[msmakouz]

Perhaps it is better to check the type, cast to the needed type, and compare? If it doesn't add a lot of code.

[aquaminer]

i don't have ideas for testing this functionality. Poor test ⬆️

[roxblnfk]

Thank you!