My first thought is that it probably is not getting indexed in solr. That runs
in a background task, so during tests it won't happen automatically. Try adding
this:
ThreadLocalORMSession.flush_all()
M.MonQTask.run_ready()
ThreadLocalORMSession.flush_all()
The run_ready() call will run the background tasks that are queued up. The
flush_all() lines might not be necessary, but I see them in other tests. You
can try with and without them. What they do is ensure all changes to models are
flushed to mongo.
Let me know if that doesn't help and I'll take a closer look.
On 7/13/18 8:41 AM, Deshani Geethika wrote:
> Hi Dave,
>
> I have started to create a test case for Tickets Section.
>
> I was trying to create a new ticket and check whether it appears in
> Dashboard, but it doesn't work. It seems like I've done something wrong
> when creating new ticket.
>
> Could you kindly take a look at my implementation
> <https://forge-allura.apache.org/u/deshani/allura-personal-dashboard/ci/56062d5c56aad503e022727b0dd5b67d37133b2e/>
> and help me to sort out the issue?
>
> Thanks!
>
> On Thu, Jul 12, 2018 at 10:04 PM Deshani Geethika <deshanigeeth...@gmail.com>
> wrote:
>
>> Thanks. I'll start working on it and give you updates
>>
>> On Thu, Jul 12, 2018 at 8:57 PM Dave Brondsema <d...@brondsema.net> wrote:
>>
>>> I think it would be good to have a test of each of the sections. At least
>>> tickets & merge requests. For projects and activity, they are very
>>> similar &
>>> re-use code from the profile sections. So you could either copy the test
>>> structure and have very similar tests to the user profile tests, or omit
>>> the
>>> tests for those.
>>>
>>> For tickets & merge requests I think good tests would create a few
>>> tickets (or
>>> merge requests) and then get the /dashboard URL and assert that they
>>> showed up.
>>> The ForgeTracker suite should have plenty of examples of making tickets
>>> without
>>> having to do mocking probably. To make a merge request,
>>> allura.tests.model.test_repo.TestMergeRequest has an abstract test that
>>> uses
>>> some mocking. It'd be better though to use a more real merge request like
>>> ForgeGit's tests do.
>>>
>>>
>>> On 7/11/18 1:06 PM, Deshani Geethika wrote:
>>>> Hi Dave,
>>>>
>>>> I've created a test case for Dashboard sections and added a merge
>>> request
>>>> <https://forge-allura.apache.org/p/allura/git/merge-requests/268/>.
>>> Please
>>>> review it and let me know if any improvements required.
>>>>
>>>> Also, could you let me know what are the other functionalities should be
>>>> tested in Dashboard?
>>>>
>>>> Regards!
>>>>
>>>> On Sat, Jul 7, 2018 at 1:55 AM Dave Brondsema <d...@brondsema.net>
>>> wrote:
>>>>
>>>>> Hi.
>>>>>
>>>>> On profile pages, the self.project refers to that user-project
>>> /u/brondsem
>>>>> for
>>>>> example. You can get a reference to the same project in the dashboard
>>> with
>>>>> c.user.private_project()
>>>>>
>>>>> On another note, the title "My Followers" sounds like it would be
>>> people
>>>>> who
>>>>> follow me, which isn't correct. On profile pages (if you click to view
>>>>> all) it
>>>>> says "Activity from people you follow" which is lengthy but better. Or
>>>>> just
>>>>> "Activity" would be ok.
>>>>>
>>>>> -Dave
>>>>>
>>>>> On 7/6/18 3:13 PM, Deshani Geethika wrote:
>>>>>> Hi Dave,
>>>>>>
>>>>>> I've started to work on Followers Section and pushed the current code
>>> to
>>>>> my
>>>>>> fork. (
>>>>>
>>> https://forge-allura.apache.org/u/deshani/allura-personal-dashboard/ci/6522ff400387f1b1143d27cdd3afd1a63a811785/
>>>>>> )
>>>>>>
>>>>>> I need to create activity_app instance in class FollowersSection in
>>>>>> allura/ext/personal_dashboard/dashboard_main.py. (Line #163).
>>>>>>
>>>>>> In class ForgeActivityProfileSection which is in
>>> forgeactivity/main.py,
>>>>> the
>>>>>> activity_app instance is created by accessing project instance as
>>>>>> self.activity_app
>>>>>> = self.project.app_instance('activity'). (Line #278)
>>>>>>
>>>>>> How can I create activity_app instance in dashboard without a project
>>>>>> instance?
>>>>>>
>>>>>> Regards!
>>>>>>
>>>>>> On Mon, Jul 2, 2018 at 5:21 PM Deshani Geethika <
>>>>> deshanigeeth...@gmail.com>
>>>>>> wrote:
>>>>>>
>>>>>>> Hi Dave,
>>>>>>>
>>>>>>> I've completed the Tickets section and added a merge request
>>>>>>> <https://forge-allura.apache.org/p/allura/git/merge-requests/264/>
>>>>>>>
>>>>>>> Please review it and let me know if any improvements required.
>>>>>>>
>>>>>>> Regards!
>>>>>>>
>>>>>>> On Fri, Jun 29, 2018 at 8:48 PM Deshani Geethika <
>>>>>>> deshanigeeth...@gmail.com> wrote:
>>>>>>>
>>>>>>>> Hi Dave,
>>>>>>>>
>>>>>>>> Thanks. I'm still working on it. Will give you an update soon.
>>>>>>>>
>>>>>>>> Regards!
>>>>>>>>
>>>>>>>> On Fri, Jun 29, 2018 at 7:45 PM Dave Brondsema <d...@brondsema.net>
>>>>>>>> wrote:
>>>>>>>>
>>>>>>>>> Hey Deshani,
>>>>>>>>>
>>>>>>>>> Just wondering how things are going? If you want to push your
>>> current
>>>>>>>>> branch,
>>>>>>>>> and let me know if any questions or issues, I can offer some more
>>>>> help.
>>>>>>>>>
>>>>>>>>> I am impressed with how much you've figured out on your own. I'm
>>> here
>>>>>>>>> to mentor
>>>>>>>>> and help though too :)
>>>>>>>>>
>>>>>>>>> -Dave
>>>>>>>>>
>>>>>>>>> On 6/27/18 12:07 AM, Deshani Geethika wrote:
>>>>>>>>>> Thanks. I'll be there today at 11am EDT.
>>>>>>>>>>
>>>>>>>>>> On Tue, Jun 26, 2018 at 9:18 PM Dave Brondsema <
>>> d...@brondsema.net
>>>>>>>>>> <mailto:d...@brondsema.net>> wrote:
>>>>>>>>>>
>>>>>>>>>> Thanks, I've taken a look at the code to get familiar with it
>>> and
>>>>>>>>> have some
>>>>>>>>>> ideas.
>>>>>>>>>>
>>>>>>>>>> Today I am available today much any time from now until 6pm
>>> EDT.
>>>>>>>>> Or tomorrow
>>>>>>>>>> 10:30-12:00 and 1pm-6pm EDT. What do you prefer? We can
>>> switch
>>>>>>>>> over to chat to
>>>>>>>>>> coordinate too.
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> On 6/26/18 4:04 AM, Deshani Geethika wrote:
>>>>>>>>>> > Hi Dave,
>>>>>>>>>> >
>>>>>>>>>> > I have pushed the current code into my fork.
>>>>>>>>>> >
>>>>>>>>>> (
>>>>>>>>>
>>>>>
>>> https://forge-allura.apache.org/u/deshani/allura-personal-dashboard/ci/0fe5a7d86e06283801cc7c21df70ba4dfa15927b/
>>>>>>>>> )
>>>>>>>>>> >
>>>>>>>>>> > Please let me know a convenient slot to have a call.
>>>>>>>>>> >
>>>>>>>>>> > Regards!
>>>>>>>>>> >
>>>>>>>>>> > On Tue, Jun 26, 2018 at 6:49 AM Dave Brondsema <
>>>>>>>>> d...@brondsema.net
>>>>>>>>>> <mailto:d...@brondsema.net>
>>>>>>>>>> > <mailto:d...@brondsema.net <mailto:d...@brondsema.net>>>
>>>>> wrote:
>>>>>>>>>> >
>>>>>>>>>> > Hey Deshani, do you want to do another video call? If
>>> you
>>>>>>>>> want to
>>>>>>>>>> push your
>>>>>>>>>> > current code to a branch then I could take a look at it
>>>>> too,
>>>>>>>>> and
>>>>>>>>>> understand what
>>>>>>>>>> > you have so far. Then we could do a video/screenshare
>>>>> call,
>>>>>>>>> continue with
>>>>>>>>>> > emails, or do chats. Whatever you prefer.
>>>>>>>>>> >
>>>>>>>>>> > -Dave
>>>>>>>>>> >
>>>>>>>>>> >
>>>>>>>>>> >
>>>>>>>>>> > --
>>>>>>>>>> > *Deshani Geethika*
>>>>>>>>>> > Undergraduate at Department of Computer Science and
>>> Engineering
>>>>>>>>>> > Faculty of Engineering - University of Moratuwa Sri Lanka
>>>>>>>>>> > LinkedIn <https://www.linkedin.com/in/deshanigeethika/> |
>>>>> GitHub
>>>>>>>>>> > <https://github.com/deshanigtk> | Mobile - +94776383034
>>>>>>>>>> >
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> --
>>>>>>>>>> Dave Brondsema : d...@brondsema.net <mailto:
>>> d...@brondsema.net>
>>>>>>>>>> http://www.brondsema.net : personal
>>>>>>>>>> http://www.splike.com : programming
>>>>>>>>>> <><
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> --
>>>>>>>>>> *Deshani Geethika*
>>>>>>>>>> Undergraduate at Department of Computer Science and Engineering
>>>>>>>>>> Faculty of Engineering - University of Moratuwa Sri Lanka
>>>>>>>>>> LinkedIn <https://www.linkedin.com/in/deshanigeethika/> | GitHub
>>>>>>>>>> <https://github.com/deshanigtk> | Mobile - +94776383034
>>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> --
>>>>>>>>> Dave Brondsema : d...@brondsema.net
>>>>>>>>> http://www.brondsema.net : personal
>>>>>>>>> http://www.splike.com : programming
>>>>>>>>> <><
>>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>> --
>>>>>>>> *Deshani Geethika*
>>>>>>>> Undergraduate at Department of Computer Science and Engineering
>>>>>>>> Faculty of Engineering - University of Moratuwa Sri Lanka
>>>>>>>> LinkedIn <https://www.linkedin.com/in/deshanigeethika/> | GitHub
>>>>>>>> <https://github.com/deshanigtk> | Mobile - +94776383034
>>>>>>>>
>>>>>>>>
>>>>>>>
>>>>>>> --
>>>>>>> *Deshani Geethika*
>>>>>>> Undergraduate at Department of Computer Science and Engineering
>>>>>>> Faculty of Engineering - University of Moratuwa Sri Lanka
>>>>>>> LinkedIn <https://www.linkedin.com/in/deshanigeethika/> | GitHub
>>>>>>> <https://github.com/deshanigtk> | Mobile - +94776383034
>>>>>>>
>>>>>>>
>>>>>>
>>>>>
>>>>>
>>>>>
>>>>> --
>>>>> Dave Brondsema : d...@brondsema.net
>>>>> http://www.brondsema.net : personal
>>>>> http://www.splike.com : programming
>>>>> <><
>>>>>
>>>>
>>>>
>>>
>>>
>>>
>>> --
>>> Dave Brondsema : d...@brondsema.net
>>> http://www.brondsema.net : personal
>>> http://www.splike.com : programming
>>> <><
>>>
>>
>>
>> --
>> *Deshani Geethika*
>> Undergraduate at Department of Computer Science and Engineering
>> Faculty of Engineering - University of Moratuwa Sri Lanka
>> LinkedIn <https://www.linkedin.com/in/deshanigeethika/> | GitHub
>> <https://github.com/deshanigtk> | Mobile - +94776383034
>>
>>
>
--
Dave Brondsema : d...@brondsema.net
http://www.brondsema.net : personal
http://www.splike.com : programming
<><
