Sylius: [Behat] [DX] Rename TableManipulator to TableReader?
pjedrzejewski
👍2
👎1
All 6 comments
TableRowAccessor ?
Arminek
on 16 Mar 2016
@Arminek Are we sure it will only access rows? Perhaps we want to check table header, etc. Separate service just for that would be an overkill imho.
pjedrzejewski
on 16 Mar 2016
TableAccessor seems like a most suitable name for me
pamil
on 16 Mar 2016
👍3
TableAccessor and TableChecker. Inject table accessor into table checker and voila :P
Arminek
on 16 Mar 2016
For now, I'm for TableAccessor ;)
Zales0123
on 16 Mar 2016
👍2
Can be closed, it was renamed to TableAccessor.
pamil
on 25 Mar 2016
Was this page helpful?
0 / 5 - 0 ratings
Related issues
mezoni
·
3Comments
mikemix
·
3Comments
loic425
·
3Comments
eb22fbb4
·
3Comments
reyostallenberg
·
3Comments
Most helpful comment
TableAccessorseems like a most suitable name for me