#metabrainz

/

      • iliekcomputers
        it can be, yes.
      • 2020-01-18 01808, 2020

      • iliekcomputers
        we'd have to upgrade brainzutils in AB to get the new flask version into AB, so that would be a seperate pull request.
      • 2020-01-18 01838, 2020

      • sarthak_jain
        Okay I am commiting the changes now.
      • 2020-01-18 01844, 2020

      • sarthak_jain
        You might wanna check.
      • 2020-01-18 01850, 2020

      • iliekcomputers
        hmm
      • 2020-01-18 01840, 2020

      • sarthak_jain
        Requested review
      • 2020-01-18 01845, 2020

      • sarthak_jain
        '=D
      • 2020-01-18 01858, 2020

      • sarthak_jain
        Do you have any idea why this `lastfailed` file gets created whenever I push code?
      • 2020-01-18 01836, 2020

      • iliekcomputers
        i am not sure
      • 2020-01-18 01836, 2020

      • iliekcomputers
        it is probably from the test run
      • 2020-01-18 01847, 2020

      • sarthak_jain
        Ohh
      • 2020-01-18 01856, 2020

      • iliekcomputers
        you should use `git add -p` instead of `git add .`
      • 2020-01-18 01804, 2020

      • sarthak_jain
        Okayy
      • 2020-01-18 01805, 2020

      • iliekcomputers
        so that you know which files you're adding to the commit
      • 2020-01-18 01812, 2020

      • sarthak_jain
        Yupp
      • 2020-01-18 01815, 2020

      • sarthak_jain
        makes sense.
      • 2020-01-18 01846, 2020

      • sarthak_jain
        I have deleted that file
      • 2020-01-18 01805, 2020

      • sumedh joined the channel
      • 2020-01-18 01816, 2020

      • sarthak_jain
        iliekcomputers: the function which we are talking about to delete listens should be availaible to the user, right?
      • 2020-01-18 01825, 2020

      • iliekcomputers
        yes!
      • 2020-01-18 01801, 2020

      • sarthak_jain
        Okay let me explore the codebase more
      • 2020-01-18 01811, 2020

      • sarthak_jain
        If i have any doubts I will get back to you
      • 2020-01-18 01858, 2020

      • iliekcomputers
        mhmm
      • 2020-01-18 01842, 2020

      • sarthak_jain
        Thanks :)
      • 2020-01-18 01841, 2020

      • antara has quit
      • 2020-01-18 01818, 2020

      • antara joined the channel
      • 2020-01-18 01847, 2020

      • CODEin_JDB has quit
      • 2020-01-18 01853, 2020

      • iliekcomputers
        ruaok: confirmed that the rabbitmq connection works from the new cluster
      • 2020-01-18 01825, 2020

      • ishaanshah[m]
        iliekcomputers: Hi!, should I seperate the importer logic from the component or it is fine in the same file.
      • 2020-01-18 01842, 2020

      • iliekcomputers
        ishaanshah[m]: hi!
      • 2020-01-18 01854, 2020

      • iliekcomputers
        ideally a different file per component is ok.
      • 2020-01-18 01809, 2020

      • ruaok
        iliekcomputers: fabu
      • 2020-01-18 01825, 2020

      • iliekcomputers
        you should make LastFmImporterModal.js which has just the code for the modal.
      • 2020-01-18 01816, 2020

      • ishaanshah[m]
        Ohk
      • 2020-01-18 01822, 2020

      • iliekcomputers
        (which would include the import code, I guess)
      • 2020-01-18 01808, 2020

      • ishaanshah[m]
        I thought we could have LastFmImporterModal.jsx for handling the UI and actual logic for import in a seprate JS file
      • 2020-01-18 01815, 2020

      • iliekcomputers
        Sounds very reasonable to me.
      • 2020-01-18 01840, 2020

      • ishaanshah[m]
        I'll rename the files in the end
      • 2020-01-18 01857, 2020

      • ishaanshah[m]
        So that it becomes easier to track changes using git
      • 2020-01-18 01836, 2020

      • iliekcomputers
        Sure.
      • 2020-01-18 01808, 2020

      • ishaanshah[m]
        Btw, can you help me out in this
      • 2020-01-18 01814, 2020

      • ishaanshah[m]
      • 2020-01-18 01817, 2020

      • iliekcomputers
        ishaanshah[m]: I'm really interested in tests for this part of the codebase, thanks for looking at it.
      • 2020-01-18 01828, 2020

      • iliekcomputers
        Reading rn.
      • 2020-01-18 01802, 2020

      • ishaanshah[m]
        I can't find an easy way to test the functions that I have added.
      • 2020-01-18 01811, 2020

      • ZaphodBeeblebrox is now known as CatQuest
      • 2020-01-18 01857, 2020

      • iliekcomputers
        so, ideally we should just mock the `fetch` function there and assert that it's called with the correct parameters
      • 2020-01-18 01820, 2020

      • iliekcomputers
        if you use jest (https://jestjs.io/docs/en/mock-functions) it should be easy to do.
      • 2020-01-18 01846, 2020

      • ishaanshah[m]
        Thanks, I will look into it.
      • 2020-01-18 01810, 2020

      • iliekcomputers
        hmm.
      • 2020-01-18 01812, 2020

      • iliekcomputers
        I haven't looked into it in detail but the gist should be that you'd have to add jest and enzyme to package.json, add the test files and have a npm function to run the tests
      • 2020-01-18 01850, 2020

      • ishaanshah[m]
        Ohk
      • 2020-01-18 01826, 2020

      • sarthak_jain has quit
      • 2020-01-18 01809, 2020

      • Cyna
        bitmap: is there a way to port our current selenium tests and custom commands over to the latest version so we can use browser to create selenium tests easily ?
      • 2020-01-18 01837, 2020

      • antara has quit
      • 2020-01-18 01847, 2020

      • rahul24 joined the channel
      • 2020-01-18 01848, 2020

      • bitmap
        Cyna: not atm, it'll require a ton of work
      • 2020-01-18 01850, 2020

      • Cyna
        If possible, please let me know how its possible. I'll give it some time :)
      • 2020-01-18 01857, 2020

      • rahul24_ joined the channel
      • 2020-01-18 01823, 2020

      • rahul24 has quit
      • 2020-01-18 01823, 2020

      • rahul24_ is now known as rahul24
      • 2020-01-18 01808, 2020

      • sumedh has quit
      • 2020-01-18 01801, 2020

      • c1e0 has quit
      • 2020-01-18 01809, 2020

      • rahul24_ joined the channel
      • 2020-01-18 01813, 2020

      • rahul24 has quit
      • 2020-01-18 01813, 2020

      • rahul24_ is now known as rahul24
      • 2020-01-18 01811, 2020

      • reosarevok
        bitmap, yvanzo: https://github.com/metabrainz/musicbrainz-server/… now open since the release one got merged
      • 2020-01-18 01802, 2020

      • bitmap
        Cyna: I haven't thought too much about the best way to solve this, I just know it'll be a lot of work :) it's a better use of time to work with what we have (after all, it's not impossible to write tests now) and get your existing work merged
      • 2020-01-18 01839, 2020

      • Cyna
        bitmap: could you help me with this ?
      • 2020-01-18 01840, 2020

      • Cyna
      • 2020-01-18 01802, 2020

      • Cyna
        I've run the command `sudo ./t/selenium.js --headless`
      • 2020-01-18 01846, 2020

      • bitmap
        it shouldn't be run with sudo, actually, and I imagine that can cause issues as the local::lib env variables won't be set in the sudo environment
      • 2020-01-18 01818, 2020

      • bitmap
        unless you use -E maybe, but you still shouldn't use sudo there
      • 2020-01-18 01831, 2020

      • Cyna
        I've tried running without sudo... then I'm getting permission issues from chromedriver
      • 2020-01-18 01852, 2020

      • Cyna
        WebDriverError: <unknown>: Failed to read the 'sessionStorage' property from 'Window': Access is denied for this document.
      • 2020-01-18 01834, 2020

      • Cyna
        well using -E didnt work
      • 2020-01-18 01836, 2020

      • bitmap
        that looks unrelated to any file permission issue
      • 2020-01-18 01852, 2020

      • bitmap
        that error is internal to your browser, not anything to do with your system
      • 2020-01-18 01828, 2020

      • Cyna
        Im using chrome
      • 2020-01-18 01847, 2020

      • Cyna
      • 2020-01-18 01856, 2020

      • Cyna
        this is the complete log
      • 2020-01-18 01839, 2020

      • bitmap
        never saw that before, but I googled the error message and found https://stackoverflow.com/a/26671889
      • 2020-01-18 01846, 2020

      • bitmap
        try adjusting your chrome settings
      • 2020-01-18 01817, 2020

      • Cyna
        cant find that specific setting as mentioned in stackoverflow
      • 2020-01-18 01838, 2020

      • rdswift_lt joined the channel
      • 2020-01-18 01850, 2020

      • bitmap
        I'm not sure, sorry
      • 2020-01-18 01824, 2020

      • yvanzo
        bitmap, reosarevok: do we release next version on Monday?
      • 2020-01-18 01835, 2020

      • reosarevok
        Have we had anything on beta?
      • 2020-01-18 01813, 2020

      • yvanzo
      • 2020-01-18 01830, 2020

      • yvanzo
        (plus probably some untracked changes)
      • 2020-01-18 01828, 2020

      • reosarevok
        I mean, I guess it'd make sense to keep the schedule, so ok
      • 2020-01-18 01852, 2020

      • bitmap
        we can release those, sure
      • 2020-01-18 01853, 2020

      • reosarevok
        bitmap: yvanzo is ok with https://github.com/metabrainz/musicbrainz-server/… so feel free to merge it
      • 2020-01-18 01855, 2020

      • yvanzo
        from git log, there also is dev doc change and that’s it.
      • 2020-01-18 01837, 2020

      • bitmap
        but I'd like to include MBS-10571 and a fix to load the countries domain on some pages, both of which I'll submit today
      • 2020-01-18 01838, 2020

      • BrainzBot
        MBS-10571: Localized modbot notes are not properly formatted when sent via email https://tickets.metabrainz.org/browse/MBS-10571
      • 2020-01-18 01855, 2020

      • yvanzo
        +1
      • 2020-01-18 01815, 2020

      • bitmap
        (the i18n error doesn't have a ticket afaik but it's on sentry)
      • 2020-01-18 01859, 2020

      • Cyna
        bitmap: the problem was that the plackup was not active in the background. I was assuming that plackup is not required
      • 2020-01-18 01805, 2020

      • bitmap
        ah ok. yes it's required, it just runs the selenium tests against your local server but doesn't start anything for you
      • 2020-01-18 01854, 2020

      • Cyna
        Is there a way for me to remove an account from the db without having to reset it ?
      • 2020-01-18 01835, 2020

      • Cyna
        When running the selenium suite again I'm getting user already exist, because it was created initially and wasnt removed
      • 2020-01-18 01813, 2020

      • reosarevok
        bitmap: if you send it in the next couple hours max I can review it today
      • 2020-01-18 01818, 2020

      • reosarevok
        (I'm going for a hike tomorrow)
      • 2020-01-18 01850, 2020

      • bitmap
        reosarevok: I'll try, we can do a late release on monday/early tuesday if not
      • 2020-01-18 01854, 2020

      • reosarevok
        Ok
      • 2020-01-18 01807, 2020

      • reosarevok
        I guess Monday review is fine too, actually :D
      • 2020-01-18 01811, 2020

      • reosarevok
        Forgot about that
      • 2020-01-18 01802, 2020

      • bitmap
        Cyna: the selenium.js script should clear the db for you
      • 2020-01-18 01819, 2020

      • bitmap
        if it's not working you can drop/recreate it
      • 2020-01-18 01846, 2020

      • bitmap
        but I haven't had an issue with it leaving users behind
      • 2020-01-18 01813, 2020

      • rahul24
        Hi everyone, in Picard, is there a way to know if it is taking the fingerprint of an audio file or not? I am trying to take the fingerprint of a larger file and so I don’t know if it is taking the fingerprint
      • 2020-01-18 01813, 2020

      • reosarevok
        bitmap: it might be one of those cases where if it crashes it doesn't clean up
      • 2020-01-18 01826, 2020

      • reosarevok
        I remember hitting one of those, maybe even with selenium
      • 2020-01-18 01802, 2020

      • Cyna
        why is it necessary to run `./script/compile_resources.sh` from create_test_db.sh ?
      • 2020-01-18 01809, 2020

      • bitmap
        that's probably possible, though it should clean it when you run selenium.js again (before running any tests), so a bit confused why that's not working
      • 2020-01-18 01809, 2020

      • bitmap
        Cyna: from?
      • 2020-01-18 01832, 2020

      • bitmap
        (what do you mean by from)
      • 2020-01-18 01858, 2020

      • rdswift_lt
      • 2020-01-18 01806, 2020

      • Cyna
        sorry I meant after
      • 2020-01-18 01813, 2020

      • rdswift_lt
        That might give you some ideas.
      • 2020-01-18 01819, 2020

      • Cyna
        I guess im just too sleepy to type anything
      • 2020-01-18 01839, 2020

      • Cyna
        My register user selenium is still failing, I tried checking using --headless=false and the user is already present even though I reset the database
      • 2020-01-18 01840, 2020

      • Cyna
        :(
      • 2020-01-18 01815, 2020

      • Cyna
        bitmap: ^
      • 2020-01-18 01832, 2020

      • Cyna
        I'll come back tomorrow. Good night !
      • 2020-01-18 01834, 2020

      • rahul24
        Thank you! I will take a look at that
      • 2020-01-18 01829, 2020

      • rdswift_lt
        rahul24, you might also be able to use a fingerprinter from https://acoustid.org/fingerprinter
      • 2020-01-18 01803, 2020

      • rahul24
        Ah right, I forgot about those, I’ll try downloading them. Thanks again :D
      • 2020-01-18 01805, 2020

      • bitmap
        Cyna: those scripts aren't related (so the order doesn't matter). it suggests running compile_resources.sh again just to make sure the tests are running against the current code. if you're sure it's current (or are using --watch), you don't need to
      • 2020-01-18 01816, 2020

      • bitmap
        the only idea I have for the user thing is to drop musicbrainz_test and run create_test_db again
      • 2020-01-18 01839, 2020

      • bitmap
        maybe it was present in the test db before you created musicbrainz_selenium, so the template wasn't clean
      • 2020-01-18 01806, 2020

      • bitmap
        of course you'll need to drop musicbrainz_selenium again and recreate that afterward, too
      • 2020-01-18 01856, 2020

      • bitmap
        let me know if that helps tomorrow. sleep well!
      • 2020-01-18 01819, 2020

      • rahul24
        In BookBrainz, when you are adding an author to a book, is the relationship “edited”? I don’t see a simple author option which is why I ask
      • 2020-01-18 01812, 2020

      • rdswift_lt
        I don't think "edited" is correct, but I'm not at all familiar with BookBrainz. I do know that, in the publishing world, editor is different from author.
      • 2020-01-18 01819, 2020

      • reosarevok
        Is there "wrote"?
      • 2020-01-18 01827, 2020

      • reosarevok
        If so, I'd expect that
      • 2020-01-18 01838, 2020

      • rdswift_lt
        I agree.
      • 2020-01-18 01857, 2020

      • awalvie[m]
        Hi there, does metabrainz have plans for participating in GSOC 2020 ? If yes, can someone please direct me to the relevant ideas page associated with it.
      • 2020-01-18 01808, 2020

      • rahul24
        I got the “wrote” option after adding it directly from the author page :)
      • 2020-01-18 01851, 2020

      • rahul24
        awalvie[m]: https://community.metabrainz.org/c/metabrainz/gso… I think this is where MB does GSoC applications, though I dont think they have updated it for 2020
      • 2020-01-18 01819, 2020

      • rdswift_lt
        !recall gsoc
      • 2020-01-18 01819, 2020

      • BrainzBot
      • 2020-01-18 01850, 2020

      • reosarevok
        awalvie[m]: there's no page for 2020 yet, but that should give you an idea ^
      • 2020-01-18 01855, 2020

      • reosarevok
        I'd be very surprised if we didn't apply
      • 2020-01-18 01811, 2020

      • awalvie[m]
        ohhh bots, cool !
      • 2020-01-18 01831, 2020

      • rdswift_lt
      • 2020-01-18 01843, 2020

      • rdswift_lt
        !forget gsoc
      • 2020-01-18 01844, 2020

      • BrainzBot
        What was "gsoc" all about?