Problems with TestContainers and Kotlin #318

Activity

1. martin-g commented on Mar 31, 2017

   

   martin-g

   on Mar 31, 2017

   Last edited by martin-g

   More actions

   My workaround is:

   class KGenericContainer(imageName: String) : GenericContainer<KGenericContainer>(imageName)

   And use it everywhere I'd need otherwise to use GenericContainer.
   Charming, isn't it ? :-)

   👍React with 👍86Reacted by Inaiat Henrique, Anders, Arnab Mitra, Nikolay Kuznetsov, Nikita Salnikov-Tarnovski, Todd Ginsberg, Christian Helmbold, Mikael Gueck, Marcos García, Will Lokes and 76 more

   😄React with 😄8Reacted by Viacheslav Blinov, Mario Kapusta, rhoehmann, Even Lislebø, obvionaoe, João Freitas, Bohdan and QUSHLEE

   🎉React with 🎉18Reacted by Philipp Hauer, Pete Cornish, Arnab Mitra, Justin Tormey, Sämir Hamadi, Tjeu Kayim, Jim Moore, Sean Hodges, Xavier Arias Seguí, Viacheslav Blinov and 8 more

   ❤️React with ❤️16Reacted by Tjeu Kayim, Xavier Arias Seguí, Christian Dräger, Viacheslav Blinov, Mike Emery, Leon Felipe Cordero, Dmitriy Varygin, João Freitas, Nicole Penn, Bohdan and 6 more
2. rnorth commented on Apr 1, 2017

   

   rnorth

   on Apr 1, 2017

   Member

   More actions

   Yeah, I'd not appreciated this would be a problem with Kotlin but it's unfortunate :(
   @bsideup, something to consider for the next version...
3. asafm commented on Apr 1, 2017

   

   asafm

   on Apr 1, 2017 via email

   Contributor

   More actions

   Just reading this generic expression gives you a headache :)

   …

   On Sat, 1 Apr 2017 at 20:09 Richard North ***@***.***> wrote: Yeah, I'd not appreciated this would be a problem with Kotlin but it's unfortunate :( @bsideup <https://github.com/bsideup>, something to consider for the next version... — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#318 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AA8Y8Rglpfbh-GydCPzdE7d_W1nkn8FMks5rroTOgaJpZM4Mvb7T> .

   👍React with 👍14Reacted by Nikolay Kuznetsov, Christian Helmbold, Mikael Gueck, Maxim Pavlov, Daniel Leicht, Florian Hintermeier, Achraf, Hunter Sherman, vach, Tom Hermann and 4 more

   😄React with 😄1Reacted by José Vieira Neto
4. mpecan commented on Jul 18, 2017

   

   mpecan

   on Jul 18, 2017

   More actions

   This is related directly to #238 and can be resolved in the way that was suggested by @bsideup (#238 (comment)).
5. mkobit commented on May 16, 2018

   

   mkobit

   on May 16, 2018

   More actions

   Another ugly way to handle this as described in the YouTrack issue:

   val container: GenericContainer<*> = GenericContainer<Nothing>()

   👍React with 👍19Reacted by Viktor Isaev, Dan McCabe, Boon at Shift, Joonhyeok Im, Vitaly Chirkov, Viacheslav Blinov, mstrdev, Andreas Berger, Martin Vysny, Bruno Bottazzini and 9 more
6. helmbold commented on May 29, 2018

   

   helmbold

   on May 29, 2018

   More actions

   @mkobit Method chaining as shown in the docs is not possible with your approach ( GenericContainer<Nothing>()).

   I suggest to use the workaround from @martin-g instead.

   👍React with 👍2Reacted by Tim van der Leeuw and Andrey Somov
7. kiview commented on Oct 11, 2018

   

   kiview

   on Oct 11, 2018

   Member

   More actions

   First time I tried to use Testcontainers with Kotlin and ran into exactly this problem, so great it found the workarounds here 😅

   👍React with 👍18Reacted by Joel Handwell, Luis Miguel Mejía Suárez, Artem Yakshin, Michel Krämer, Cody Moore, Jim Moore, Dan McCabe, James Millner, Vitaly Chirkov, Maxim Pavlov and 8 more
8. kiview

   added

   type/feature

   resolution/acknowledged

   on Oct 11, 2018
9. alex0ptr commented on Oct 10, 2019

   

   alex0ptr

   on Oct 10, 2019

   More actions

   @helmbold Chaining is easily possible with Kotlin native scoping functions, example:

   val container: GenericContainer<*> = GenericContainer<Nothing>()
                                           .apply{ withExposedPorts(6379) }

   I'll make a merge request to propose this in the Kotlin Examples.

   👍React with 👍19Reacted by Richard North, Babur, Maksim Kostromin, Luca Piccinelli, Andrea Arcuri, João Freitas, Suresh, Sunny Pelletier, ser0l, Alex Sherwin and 9 more
10. xyzwps commented on Oct 29, 2019

    

    xyzwps

    on Oct 29, 2019

    Last edited by xyzwps

    More actions

    I created a sub class of MySQLContainer:

    internal class SpecifiedMySQLContainer(val image: String) : MySQLContainer<SpecifiedMySQLContainer>(image)
    
    val mysqlContainer = SpecifiedMySQLContainer(image = "mysql:5.7.28").withDatabaseName("dbname")

    👍React with 👍12Reacted by Josh Rotenberg, Keegan Witt, Thomas Deville-Duc, Sam, Broc Seib, Bengt Brodersen, Steve Smith, Russell J.B., Luiz Aguiar, Daniel André da Silva and 2 more

    🎉React with 🎉3Reacted by Russell J.B., Daniel André da Silva and dzikoysk
11. thaibt commented on Apr 23, 2021

    

    thaibt

    on Apr 23, 2021

    More actions

    Another ugly way to handle this as described in the YouTrack issue:

    val container: GenericContainer<*> = GenericContainer<Nothing>()

    This saved my life... Was to frustrated and I did not find this thread.

    👍React with 👍3Reacted by Dmitry Bolotin, Matt Sponer and Yuri
12. schnapster commented on Oct 22, 2021

    

    schnapster

    on Oct 22, 2021

    More actions

    Looks like Kotlin 1.5.30 ships with a preview for 1.6 that fixes this: https://kotlinlang.org/docs/whatsnew1530.html#improvements-to-type-inference-for-recursive-generic-types

    👍React with 👍3Reacted by Kevin Wittek, Erwin Wessels and Oliver Weiler
13. sdeleuze commented on Dec 28, 2021

    

    sdeleuze

    on Dec 28, 2021

    More actions

    Please close this issue as Kotlin 1.6 which fixes it has been released.

    👍React with 👍1Reacted by Sergei Egorov
14. bsideup commented on Dec 28, 2021

    

    bsideup

    on Dec 28, 2021

    Member

    More actions

    @sdeleuze good point!
    It is great to see Kotlin supporting more and more "in the wild" cases 🎉
    IIRC some Spring's builders also benefited from it? Double win!

    👍React with 👍1Reacted by Suresh
15. bsideup

    closed this as completedon Dec 28, 2021
16. sdeleuze commented on Dec 28, 2021

    

    sdeleuze

    on Dec 28, 2021

    More actions

    Yep indeed that fixed a few Spring builder APIs as well. I am glad the Kotlin team has listened feedback from the community to fix this really annoying issue.
17. sdeleuze commented on Mar 12, 2025

    

    sdeleuze

    on Mar 12, 2025

    More actions

    @asomov I just tried val container: GenericContainer<*> = GenericContainer("someImage") with Kotlin 2.1.10 and it seems to work as expected. If you manage to reproduce, could you please share a repro?

    👍React with 👍3Reacted by Eddú Meléndez Gonzales, Suresh and Andrey Somov