talawa icon indicating copy to clipboard operation
talawa copied to clipboard

Unittests for widget video_widget.dart

Open palisadoes opened this issue 2 years ago • 15 comments

  • Please coordinate issue assignment and PR reviews with the contributors listed in this issue https://github.com/PalisadoesFoundation/talawa/issues/359

We will need unittests done for all methods, classes and functions found in this file.

Any widgets referenced in this file must also have unittests done

video_widget.dart

This file can be found in the lib/widgets directory

Parent Issue:

  • https://github.com/PalisadoesFoundation/talawa/issues/1125

PR Acceptance Criteria

  • When complete this file must show 100% coverage when merged into the code base.
  • The current code coverage for the file can be found here
  • The PR will show a report for the code coverage for the file you have added. You can use that as a guide
  • No functionality must be removed to achieve this goal

palisadoes avatar Jan 16 '22 18:01 palisadoes

Can I work on this?

architsangal avatar Feb 23 '22 15:02 architsangal

Are you still working on this?

palisadoes avatar Mar 04 '22 15:03 palisadoes

No. I am working on other issues for now.

architsangal avatar Mar 04 '22 16:03 architsangal

can you assign this issue to me?

iamworrell avatar Jun 07 '22 23:06 iamworrell

Hey @palisadoes is @iamworrell still working on this? If not, can you please assign me this

shatanikmahanty avatar Dec 20 '22 15:12 shatanikmahanty

No, I am not currently working on it.

On Tue, Dec 20, 2022 at 10:10 AM Shatanik Mahanty @.***> wrote:

Hey @palisadoes https://github.com/palisadoes is @iamworrell https://github.com/iamworrell still working on this? If not, can you please assign me this

— Reply to this email directly, view it on GitHub https://github.com/PalisadoesFoundation/talawa/issues/1142#issuecomment-1359538198, or unsubscribe https://github.com/notifications/unsubscribe-auth/AXTUTOKK55O7JQ33S4U25CDWOHD4VANCNFSM5MDBQQEA . You are receiving this because you were mentioned.Message ID: @.***>

iamworrell avatar Jan 04 '23 15:01 iamworrell

@shatanikmahanty Do you still want to work on this?

palisadoes avatar Jan 24 '23 02:01 palisadoes

I am facing an issue with testing future builder widget, I was trying to find documentation on that but couldn't find proper ones that fit these use cases. If someone wants to collaborate with me for this so that I can workout the problems with future builder, then I can still put up a PR. Please wait for 1 more week, until then if I can't find a solution, I will unassign myself

shatanikmahanty avatar Jan 24 '23 03:01 shatanikmahanty

@shatanikmahanty Please contact @TheHazeEffect (Troy Anderson) on slack. He may be able to assist.

palisadoes avatar Jan 24 '23 03:01 palisadoes

@shatanikmahanty Please contact @TheHazeEffect (Troy Anderson) on slack. He may be able to assist.

Thanks for your guidance I will connect with him soon to discuss this

shatanikmahanty avatar Jan 24 '23 14:01 shatanikmahanty

@shatanikmahanty Any update ?

noman2002 avatar Feb 09 '23 20:02 noman2002

Hi @noman2002, as @palisadoes instructed I contacted Troy Anderson on Slack. I am currently awaiting a reply from his end.

shatanikmahanty avatar Feb 10 '23 07:02 shatanikmahanty

@TheHazeEffect please provide feedback

palisadoes avatar Feb 10 '23 13:02 palisadoes

This issue did not get any activity in the past 10 days and will be closed in 180 days if no update occurs. Please check if the develop branch has fixed it and report again or close the issue.

github-actions[bot] avatar Jan 02 '24 00:01 github-actions[bot]

This issue did not get any activity in the past 10 days and will be closed in 180 days if no update occurs. Please check if the develop branch has fixed it and report again or close the issue.

github-actions[bot] avatar Feb 16 '24 00:02 github-actions[bot]