Is this cool for me to do?
| Comment | File | Size | Author |
|---|---|---|---|
| #313 | 10.1-3347291-combine-mega-e.patch | 303.75 KB | srishtiiee |
| #312 | 10.1-3347291-combine-mega-d.patch | 304.04 KB | srishtiiee |
| #311 | 10.1-3347291-combine-mega-c.patch | 303.7 KB | tedbow |
| #310 | 10.1-3347291-combine-mega-b.patch | 304.04 KB | tedbow |
| #309 | 10.1-3347291-combine-mega-a.patch | 304.12 KB | tedbow |
Issue fork drupal-2977515
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
- 2977515-pm-core-mr-9-2023
changes, plain diff MR !4730
- 3346707-package-manager
changes, plain diff MR !5011
- 3253158-auto-updates-alpha
changes, plain diff MR !5881
- 2977515-march10
changes, plain diff MR !3622
- 2977515-march3
changes, plain diff MR !3578
- 2977515-package-manager-mr
changes, plain diff MR !3476
- 2977515-general-reviewe
compare
- 2977515-ignore-tedbow-fork
changes, plain diff MR !298
- 10.1-3347291-combine-forms-mega
compare
- 10.1.x
compare
- 11.x
compare
- 9.4.x
compare
Comments
Comment #2
tedbowwhats up with this test?
Comment #3
tedbowComment #5
tedbowif ignore message?
Comment #7
tedbowwhat is page text at this point?
Comment #9
tedbowthis?
Comment #10
tedbow🎉
removing some of the other changes
Comment #11
tedbowComment #12
tedbowwhy so random?
Comment #14
tedbow😿
Comment #15
tedbow⛵️
Comment #17
tedbowis access on default the problem?
Comment #19
tedbowif in_preview don't check dependency access
Comment #21
tedbow🎭
Comment #23
tedbow🙀
Comment #25
tedbow⏰
Comment #27
tedbow😅
Comment #28
tedbowComment #30
tedbow115, whoops
Comment #32
tedbow∞🤷♂️
Comment #34
tedbowdrupalGet()
Comment #36
MixologicI was working on something else and noticed this patch with only 2 results, and stopped by to look at it.
FYI: you dont have to add all that code to run-tests.sh anymore to get it to run just the one test, you can add arguments to drupalci.yml to make it do that now:
Example of how I did that in another patch: https://www.drupal.org/project/drupal/issues/2942900#comment-12620690
(the times 50 patch)
(other details here: https://www.drupal.org/drupalorg/docs/drupal-ci/customizing-drupalci-tes...)
Comment #37
tedbow@Mixologic thank for letting me know this sounds like a great change!
Comment #40
tedbowAdding a wait for "Save Layout"
changing to use method @Mixologic mentioned above. Thanks!!!!!!!!
Comment #42
tedbow🐛
Comment #44
tedbow🐼
Comment #46
tedbow🐲
Comment #48
tedbow🌋
Comment #49
tedbow🏔
Comment #51
tedbowTest to see how many dataProviders have duplicate data sets.
Comment #53
tedbowTaking a fix to ContextualLinkClickTrait.php from #2924201: Resolve random failure in LayoutBuilderTest so that it can be added to HEAD
to see if
clickLink()will work for [32968110]Comment #55
tedbowtrying to see if
waitForElementVisible()sometime is not enough to avoid random error of clicking on linkComment #57
tedbowComment #59
tedbowtrying without fix but no animation
Comment #62
tacituseu commentedDon't mind me, didn't want to start another one, should be just this one patch.
Comment #64
tedbowI think this early return statement will cause the rest of the function not to run 😜
Anyways.....
Another experiment
Comment #66
tacituseu commentedI sure hoped so ;), it was just a proof patch for what ended up as #3032869: \Drupal\Core\Update\UpdateKernel::fixSerializedExtensionObjects() caused test runs to double in duration.
Comment #67
tedbowhow to get test output??
Comment #69
tacituseu commentedAccording to drupalci_testbot and
BrowserHtmlDebugTrait::htmlOutput()it should be at artifacts/run_tests.javascript/simpletest_html/.Comment #70
tedbow@tacituse LOL thanks for the tip.
Comment #72
tacituseu commentedSometimes I also found helpful spamming the following:
$this->createScreenshot(\Drupal::root() . '/sites/default/files/simpletest/' . date('YmdHis') . microtime(true) . '-before-click.jpg');around the breaking steps (especially having to do with animations/JS), they'll end up in
phpunit-xml/subdir, nothing beats visual.Comment #73
tedbowComment #75
tedbowComment #77
tedbowassertEntityInstanceStates add wait
Comment #79
tedbowoqoqoqoqoq
Comment #81
tedbowComment #83
tedbowComment #85
tedbowComment #87
tedbowComment #89
tedbowComment #91
tedbowComment #93
tedbowComment #95
tedbowComment #97
tedbowComment #99
tedbowComment #101
tedbowComment #103
tedbowis this actually a quickedit problem?
Comment #105
tedbowComment #107
tedbowtesting to see if revert happens
Comment #109
tedbowComment #111
tedbowComment #114
tedbowComment #115
tedbowComment #117
tedbowComment #118
tedbowComment #120
tedbowComment #121
tedbowComment #122
tedbowComment #123
tedbowComment #124
tedbowComment #125
tedbowComment #127
tedbowHow many times do we call installEntitySchema for config entities
Comment #129
tedbowComment #131
tedbowComment #132
tedbowComment #133
tedbowComment #135
tedbowfor #3055648: Frequent random fail in \Drupal\Tests\media_library\FunctionalJavascript\MediaLibraryTest
and #3044496-104: [ignore] bnjmnm patch testing
Comment #136
tedbow@group 😱
Comment #137
tedbowhow badly broken?
Comment #139
tedbowhttps://www.drupal.org/project/drupal/issues/2313917#comment-13195309
Comment #141
tedbowComment #142
tedbowsimpletest 😱
Comment #143
tedbowrebase
Comment #144
tedbow🤷♂️
Comment #145
tedbowComment #146
tedbowComment #147
tedbowComment #149
tedbowis it needed?
Comment #151
tedbowwill it blend?
Comment #153
tedbowmodule directories?
Comment #155
tedbowgot bool condition wrong
Comment #157
tedbowI am totally nailing this!
Comment #159
tedbow🤞😅
Comment #160
tedbowok great. lets check the other tests
Comment #161
tedbowwhat are we testing here?
Comment #163
tedbowhttps://www.drupal.org/project/drupal/issues/3095755
Comment #165
tedbowComment #166
tedbowComment #168
tedbowComment #169
tedbowphpcs lint fix?
Comment #170
tedbowassign key first?
Comment #171
tedbowescaped double quotes?
Comment #172
tedbowComment #173
tedbowRemove cases
Comment #174
tedbowsimplify
Comment #175
tedbowComment #176
tedbowComment #178
tedbowComment #179
tedbowThis passes locally. timing?
Comment #180
tedbowclass wrong
Comment #182
tedbowuse
+2 daysComment #184
tedbowuse
send()directlyComment #186
tedbowjust the problem method
Comment #188
tedbowassert the message count directly
Comment #190
tedbowComment #193
tedbowComment #195
tedbowComment #197
tedbowComment #199
tedbowThis is best use of my time
Comment #201
tedbowComment #203
tedbowabsolute path?
Comment #205
tedbowComment #207
tedbowComment #209
tedbowComment #210
tedbowComment #211
tedbowComment #213
tedbowComment #215
tedbowComment #216
tedbowComment #218
tedbowComment #220
tedbowComment #222
tedbow😡
Comment #223
tedbowComment #225
tedbowComment #228
tedbowComment #229
tedbowComment #231
tedbowComment #233
tedbowComment #234
tedbowComment #236
tedbowComment #238
tedbowComment #240
tedbowComment #242
tedbowComment #244
tedbowComment #246
tedbowComment #248
tedbowComment #250
tedbowComment #252
tedbowComment #253
tedbowComment #255
tedbowComment #257
tedbowComment #260
tedbowComment #261
tedbowComment #263
tedbowComment #264
tedbowComment #267
tedbowphpcs error
Comment #273
tedbowComment #277
sandeepsingh199 commentedComment #278
tedbow@SandeepSingh199 please don't change the project of this issue.
Also the Automatic Updates Initiative team is going to be using this issue for testing the conversion of the contrib module for core inclusion so this should be the temporary focus of this issue
Comment #285
Yogesh Sahu commentedComment #286
Yogesh Sahu commentedComment #287
adeshsharma commentedComment #288
smustgrave commentedNot closing out as I know this is in use. But part of cleaning out the review queue moving this out to not affect the metrics. Can still use it by clicking Add test/retest.
Thanks!
Comment #289
Yogesh Sahu commentedComment #291
wim leersUh oh, that is a massive conflict in
composer.lockand core's metapackages'composer.jsonfiles that is not worth our time solving.PLUS! Since Feb 15, we fixed PHPStan compliance (which was manually disabled on this MR), removed the dependencies on
symfony/finderandsymfony/config(which apparently were manually added at the start of the MR, not by the script). So … starting a new branch:2977515-march3.🤞
Comment #293
wim leersOMG.
Comment #296
tedbowComment #297
tedbowComment #298
smustgrave commentedJust cleaning up review queue.
Comment #300
tedbowtesting backporting #3347291: Combine field storage and field instance forms back to 10.1.x
Comment #302
tedbowComment #303
tedbowComment #304
tedbowComment #305
tedbowComment #306
srishtiiee commentedComment #307
srishtiiee commentedComment #308
tedbowComment #309
tedbowComment #310
tedbowComment #311
tedbowComment #312
srishtiiee commentedComment #313
srishtiiee commented