Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bring numbers of objects up to date with latest NFRs #65

Open
emteknetnz opened this issue Feb 23, 2020 · 1 comment
Open

Bring numbers of objects up to date with latest NFRs #65

emteknetnz opened this issue Feb 23, 2020 · 1 comment

Comments

@emteknetnz
Copy link
Member

The following updates will bring this module into line with our latest NFR's as specified here

This work will assist this the 'more realistic platform regression testing' issue here https://github.com/silverstripeltd/product-issues/issues/33

FTPageMakerTask.php

  • 10,000 pages total, and at least one page with >200 child pages
  • 5 levels of page hierarchy

TODO:
Hardcode to the following: (don't bother making configurable)

  • 10 (was 5)
  • 250 (was 100)
  • 1
  • 1
  • 1

FTFileMakerTask.php

  • 10,000 files, and at least one folder with >1000 files
  • Files contain a variety of types, e.g. JPEGs with 2MB file size

TODO:

  • change protected $folderCountByDepth to
    0 => 2,
    1 => 1, (was 2)
    2 => 2,
    3 => 2,
    4 => 2,

  • change protected $fileCountByDepth to
    0 => 100,
    1 => 1001, (was 30)
    2 => 5,
    3 => 5,
    4 => 5

Company.php

  • 10,000 objects (managed through ModelAdmin or GridField)

TODO:

  • update data() to go from 216 companies to 10K
  • simply that array and suffix '-01', '02' etc 50 times

TODO:
Also as a part of work, fix this issue at the same time #64

@emteknetnz emteknetnz changed the title Bring numbers of objects update to date with latest NFRs Bring numbers of objects up to date with latest NFRs Feb 23, 2020
@chillu
Copy link
Member

chillu commented Jul 27, 2020

I've raised this in #50, but your ticket is more detailed :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants